Skip to content

Add missing variants, tag v2021.09.2#22

Merged
FelicianoTech merged 2 commits into
CircleCI-Public:mainfrom
kmsbernard:add-missing-variant
Oct 1, 2021
Merged

Add missing variants, tag v2021.09.2#22
FelicianoTech merged 2 commits into
CircleCI-Public:mainfrom
kmsbernard:add-missing-variant

Conversation

@kmsbernard

@kmsbernard kmsbernard commented Sep 28, 2021

Copy link
Copy Markdown
Contributor

I found that missing variants in the manifest file: browsers and node. These variants described as available in the README. This PR adds those two variants in the manifest file.

I also added Dockerfiles with tag 2021.09.2 via script, but I'm not sure I could do this. (perhaps only for the maintainer during release?)

@kmsbernard
kmsbernard requested review from a team and FelicianoTech as code owners September 28, 2021 04:24
@FelicianoTech

Copy link
Copy Markdown
Contributor

Hi. Thank you for the PR. These changes look good and are needed so thank you. I was about to do them myself until I saw you provided them already. Just two notes:

  1. Yes we would handle the release tag ourselfs so you don't need to worry about determining the tag/version.
  2. We don't edit Dockerfiles directly. Instead, there's a Dockerfile.template file that we make changes to, then the Dockerfiles are generated from there. In this scenario I'm going to go ahead and merge your work since we're switching over to October for the tag, but just a note for the future.
    Thank you!

@FelicianoTech
FelicianoTech merged commit 35b0eea into CircleCI-Public:main Oct 1, 2021
@FelicianoTech

Copy link
Copy Markdown
Contributor

Actually, I don't see that Dockerfile.template was updated in the "Files Changes" tab on GitHub but when I look at the actual code, it was. So ignore point number 2 that I made. This PR seems to have done that part correctly.

Again, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants