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 Action to Move Current Regulations to History #1191

Merged
merged 3 commits into from
Jan 7, 2025
Merged

Conversation

FinnIckler
Copy link
Member

This should be triggered before a new regulations cycle. It can be then linked to in the history file

@FinnIckler FinnIckler requested a review from a team as a code owner January 31, 2024 13:20
@Nevseros
Copy link
Member

This does not include adding a line saying that this version is not the current one, right?

@FinnIckler
Copy link
Member Author

You are right, I'll add that tomorrow

@FinnIckler
Copy link
Member Author

@Nevseros
Copy link
Member

Nevseros commented Feb 2, 2024

@FinnIckler

it looks like it also wasn't added the last time regulations were cycled!

Because we add regulations on the history page as soon as we announce them (2024-01-01 should be there too now, for example, but I've been lazy) to be able to reference a current regulation with permalink (during investigations, for example).

Is it possible that upon adding a current version to the history the previous version file is edited with the disclaimer, but not the one that's being added? Like I did in these two PRs in total: 8183 and 8283.

@FinnIckler
Copy link
Member Author

The github action can't figure out what the current/previous version is unless you explicitly tell it through an input variable. What we could do is also add the current version to its history folder every time the official branch is updated and then have this action that just inserts the disclaimer for a given version.

@FinnIckler
Copy link
Member Author

So I can pick this one up again. What does WRC think about my suggestion about replacing the current regulations in history every time it's updated?

@Nanush7
Copy link
Member

Nanush7 commented May 24, 2024

How would the process look like from our side? We just need to update the version file in the history folder?

@FinnIckler
Copy link
Member Author

FinnIckler commented Jun 7, 2024

No, the process from your side would just be like now, you push to this repo to the official branch and that would replace both the latest version and the latest version in history. If you then need to add the disclaimer you can run this Action.

I am looking into it and I think I can't implement it here, instead the wrc compiler needs to put the current files into the history folde

I can just do it here

@FinnIckler
Copy link
Member Author

I added a step to the deploy-to-s3 action that copies the current version into the history.
When you cycle regulations you just need to:

  • use the cycle-regulations actions to add the This is NOT the current official version in there with the version you want to add it to as the input parameter
  • Make a PR to the Main Repo to add a link to the new History file

@Nanush7
Copy link
Member

Nanush7 commented Dec 26, 2024

Thank you @FinnIckler! Are we ready to merge this?

Copy link
Member

@Nanush7 Nanush7 left a comment

Choose a reason for hiding this comment

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

I don't have any way to test this, but I don't see any problem with the changes.

@dunkOnIT dunkOnIT merged commit 4f51247 into draft Jan 7, 2025
1 check passed
@dunkOnIT dunkOnIT deleted the history-action branch January 7, 2025 09:34
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.

4 participants