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

Update markdown md CI #2585

Closed
wants to merge 8 commits into from
Closed

Update markdown md CI #2585

wants to merge 8 commits into from

Conversation

sukantoraymond
Copy link
Collaborator

Changes Update markdown md CI so that it doesn't make a new commit after every push to a PR, instead it modifies the commit to include the changes to relevant md files

@github-actions github-actions bot force-pushed the update-markdown-fix branch from 5925fd0 to b24f7cc Compare January 30, 2025 23:05
sdk/SDK.md
commit-message: "chore: Update MD files [skip ci]"
ref: ${{ github.head_ref }}
- name: Check if there are changes
Copy link
Collaborator

Choose a reason for hiding this comment

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

I can see that it sets
changes=true in $GITHUB_ENV

I am not sure what it does with this

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it reads it later

  • name: Commit changes directly to PR branch
    if: env.changes == 'true'

@@ -299,7 +299,7 @@ avalanche blockchain create [subcommand] [flags]
--evm-token string token symbol to use with Subnet-EVM
--external-gas-token use a gas token from another blockchain
-f, --force overwrite the existing configuration if one exists
--from-github-repo generate custom VM binary from github repository
--from-github-repos generate custom VM binary from github repository
Copy link
Collaborator

Choose a reason for hiding this comment

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

why this change?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

its test change to see if it will be removed before this pr is merged by the CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done ✅
Development

Successfully merging this pull request may close these issues.

3 participants