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
gkalpak merged 1 commit into
angular:masterfrom
gkalpak:chore-cleanup
May 26, 2020
Merged

chore(*): clean up package.json and CircleCI config#17034
gkalpak merged 1 commit into
angular:masterfrom
gkalpak:chore-cleanup

Conversation

@gkalpak

@gkalpak gkalpak commented May 24, 2020

Copy link
Copy Markdown
Member

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

@petebacondarwin

Copy link
Copy Markdown
Contributor

Why two PRs? This and #17033

@gkalpak

gkalpak commented May 24, 2020

Copy link
Copy Markdown
Member Author

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

@petebacondarwin petebacondarwin left a comment

Copy link
Copy Markdown
Contributor

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?

Comment thread .circleci/config.yml
@@ -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
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

❤️

@gkalpak

gkalpak commented May 25, 2020

Copy link
Copy Markdown
Member Author

🤜🤛

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).

@Splaktar Splaktar left a comment

Copy link
Copy Markdown
Contributor

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