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

Add versions on readme wiht action to update #2647

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

AurelienFT
Copy link
Contributor

Description

I added the versions used in different networks for other people to easily have the information. To avoid technical debt and maintenance I added an Action that chekcs every hour if a network has been updated and creates a PR to update the table if needed.

This has been tested on a fork of chain-configuration repository.

Job that create PR : https://github.com/AurelienFT/chain-configuration/actions/runs/13032692218/job/36355422292
PR : https://github.com/AurelienFT/chain-configuration/pull/6
Job when a PR already exists : https://github.com/AurelienFT/chain-configuration/actions/runs/13032715720/job/36355503047
Job when the versions are up-to-date : https://github.com/AurelienFT/chain-configuration/actions/runs/13032730000/job/36355548982

Checklist

  • Breaking changes are clearly marked as such in the PR description and changelog
  • New behavior is reflected in tests
  • The specification matches the implemented behavior (link update PR if changes are needed)

Before requesting review

  • I have reviewed the code myself
  • I have created follow-up issues caused by this PR and linked them here

Note: We may have a problem with CLA such as we have with cargo update and so I think we need to configure CLA to allow github action user.

@AurelienFT AurelienFT changed the title Add versions on readme Add versions on readme wiht action to update Jan 29, 2025
@AurelienFT AurelienFT requested a review from a team January 29, 2025 14:03
@AurelienFT AurelienFT added the no changelog Skip the CI check of the changelog modification label Jan 29, 2025
README.md Outdated Show resolved Hide resolved
@AurelienFT AurelienFT enabled auto-merge (squash) February 2, 2025 15:44
@AurelienFT AurelienFT requested review from xgreenx and a team February 2, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Skip the CI check of the changelog modification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants