Skip to content

Test workflow using Personal Access Token on new repo#27

Merged
JonoPrest merged 2 commits into
mainfrom
js/debug-codegen-diff-workflow
Jun 14, 2024
Merged

Test workflow using Personal Access Token on new repo#27
JonoPrest merged 2 commits into
mainfrom
js/debug-codegen-diff-workflow

Conversation

@JasoonS

@JasoonS JasoonS commented Jun 11, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@JasoonS JasoonS force-pushed the js/debug-codegen-diff-workflow branch from b3cd1f9 to 9d07895 Compare June 11, 2024 18:59
@github-actions

Copy link
Copy Markdown

Here's the diff of the codegen output.🙌🧠🦜

@JasoonS JasoonS requested a review from JonoPrest June 13, 2024 08:33
Comment on lines +2 to +4
AccountType,
BigDecimal,
NftCollectionEntity,

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I used this to trigger the build - not relevant to PR

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Alphabetical

Comment thread .github/workflows/codegen_diff.yml Outdated
Comment on lines +75 to +76
echo "https://${GH_PAT}:x-oauth-basic@github.com" > ~/.git-credentials
git config --global credential.helper store

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the main change, it uses a githup personal access token now rather than an operator ssh key.

run: |
cd test-codegen-output
echo "::set-output name=url::https://github.com/Float-Capital/test-codegen-output/compare/$(git rev-parse main)...${{ steps.extract_branch.outputs.branch }}"
echo "::set-output name=url::https://github.com/enviodev/test-codegen-output/compare/$(git rev-parse main)...${{ steps.extract_branch.outputs.branch }}"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

New repo

@JasoonS

JasoonS commented Jun 13, 2024

Copy link
Copy Markdown
Contributor Author

Diff is big now because main hasn't been updated for a while!

Comment thread .github/workflows/codegen_diff.yml

@JonoPrest JonoPrest left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Epic! Nice one Jasoons

@JonoPrest JonoPrest merged commit 6158bd5 into main Jun 14, 2024
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