Skip to content
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

External developer friendly changes #90

Merged
merged 10 commits into from
Feb 12, 2024
Merged

External developer friendly changes #90

merged 10 commits into from
Feb 12, 2024

Conversation

iowillhoit
Copy link
Contributor

@iowillhoit iowillhoit commented Jan 18, 2024

Warning

Do not merge yet, additional PRs incoming
TODO: Revert ew/external-friendly branch refs

We hard coded our svc-cli-bot username/email in a few spots. Making these workflows not useable to external developers.
This PR looks up the username/password based on the Github Token being used. Additional changes are made to take advantage of this lookup.

Example of a Release working on an external plugin: https://github.com/iowillhoit/plugin-workflow-test/actions/runs/7748104245
Example of this working on an internal plugin (using secrets inherit) https://github.com/salesforcecli/testPackageRelease/actions/runs/7759564485/job/21163873201#step:8:27

@W-14809762@

Other PRs:
oclif/plugin-plugins#784
salesforcecli/plugin-trust#720
salesforcecli/plugin-template-sf-external#110
forcedotcom/source-deploy-retrieve#1226
salesforcecli/cli#1456

Copy link

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Eric Willhoit. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce Inc. Contributor License Agreement and this Pull Request will be revalidated.

@iowillhoit iowillhoit requested a review from a team as a code owner February 2, 2024 16:37
.github/actions/gitConfig/action.yml Show resolved Hide resolved
.github/workflows/githubRelease.yml Show resolved Hide resolved
.github/actions/getGithubUserInfo/action.yml Outdated Show resolved Hide resolved
.github/workflows/create-github-release.yml Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@iowillhoit iowillhoit merged commit d68cdba into main Feb 12, 2024
1 check passed
@iowillhoit iowillhoit deleted the ew/external-friendly branch February 12, 2024 15:48
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.

2 participants