-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NEW (Extension) @W-17831303@ Fix release branch creation issue #171
Conversation
@@ -60,7 +60,9 @@ jobs: | |||
- uses: actions/setup-node@v4 | |||
with: | |||
node-version: 'lts/*' # Always use Node LTS for building dependencies. | |||
- run: yarn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why we use yarn here instead of using --frozen-lockfile. Let me know if this was intentional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah for consistency, we might want to just make this yarn install --frozen-lockfile
like the others.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using --frozen-lockfile
throughout probably isn't the worst idea.
@@ -60,7 +60,9 @@ jobs: | |||
- uses: actions/setup-node@v4 | |||
with: | |||
node-version: 'lts/*' # Always use Node LTS for building dependencies. | |||
- run: yarn | |||
- name: Configure Git for private repo access | |||
run: git config --global --add url."https://${{ secrets.IDEE_GH_TOKEN }}@github.com/".insteadOf "https://github.com/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the preceding
uses: actions/checkout@v4
step in other files, we have in
with:
token: ${{ secrets.IDEE_GH_TOKEN }} # So we can access internal repo to pull shared ui components
Why is this needed in the other files? I wouldn't think is needed to just do a clone of our repo.
Do we need to have that here in this file as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed offline... we should instead try to remove all this git config update stuff... in favor of just using the token during checkout. But I believe this will require you to update the dependencies in the package.json file with
- "einstein-shared": "https://github.com/forcedotcom/salesforcedx-vscode-einstein-shared#semver:0.3.10",
+ "salesforcedx-vscode-einstein-shared": "forcedotcom/salesforcedx-vscode-einstein-shared#semver:0.3.10",
and then rerun yarn install so that it updates the yarn.lock file. Then submit those updates and see if it fixes everything for us.
@@ -82,6 +82,8 @@ jobs: | |||
with: | |||
distribution: 'temurin' | |||
java-version: '11' | |||
- name: Configure Git for private repo access | |||
run: git config --global --add url."https://${{ secrets.IDEE_GH_TOKEN }}@github.com/".insteadOf "https://github.com/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto... about the actions/checkout@v4 stuff
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if it is worth creating a composite github action that for the checkout, node setup, and yarn install that we can just share all this across all our actions as a single looking action step.
No description provided.