Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

chore(*): clean up package.json and CircleCI config #17034

Merged
merged 1 commit into from
May 26, 2020

Conversation

gkalpak
Copy link
Member

@gkalpak gkalpak commented May 24, 2020

This is a follow-up to #16915, cleaning up package.json and .circleci/config.yml and making release scripts executable.

@petebacondarwin
Copy link
Contributor

Why two PRs? This and #17033

@gkalpak
Copy link
Member Author

gkalpak commented May 24, 2020

I am testing some stuff. (Still drafts - no need to review yet 😁)

This is a follow-up to angular#16915, cleaning up `package.json` and
`.circleci/config.yml` and making release scripts executable.
@gkalpak gkalpak marked this pull request as ready for review May 25, 2020 12:20
Copy link
Contributor

@petebacondarwin petebacondarwin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome - thanks for watching my back @gkalpak.

OOC where do you find the SHAs for the docker images?

@@ -1,4 +1,4 @@
# Configuration file for https://circleci.com/gh/angular/angular
# Configuration file for https://circleci.com/gh/angular/angular.js
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️

@gkalpak
Copy link
Member Author

gkalpak commented May 25, 2020

🤜🤛

The easiest way to find the SHA (that I know of) is to let the job run and look at the Spin Up Environment step of any job that uses the executor you are interested in (e.g. see here).

Copy link
Contributor

@Splaktar Splaktar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gkalpak gkalpak merged commit 295213d into angular:master May 26, 2020
@gkalpak gkalpak deleted the chore-cleanup branch May 26, 2020 14:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants