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

workflows: fix wikiupdate and bump version to v4 #765

Merged
merged 5 commits into from
Feb 10, 2024
Merged

Conversation

FFHener
Copy link
Contributor

@FFHener FFHener commented Feb 6, 2024

Move the wikiupdate-workflow into a independent file making it way more clean. And it is working now as intended (instead of building all locations) as you can see here
with a changed location
without a changed location

I use ||true so the task will never fail because there might be locations that doesn't get updated because they don't have a wikiarticle.

In addition i bumped the version of actions/checkout@v3 to v4 as the old one was using Node.js 16 which is deprecated.

Copy link
Member

@Noki Noki left a comment

Choose a reason for hiding this comment

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

We should always build configs, not just on pull requests. We should store the configs as artifacts because we don't build images anymore.

@FFHener FFHener force-pushed the testbuild-wikiupdater branch from 3d4f132 to 8f4c2ea Compare February 8, 2024 11:38
@FFHener FFHener force-pushed the testbuild-wikiupdater branch from 8f4c2ea to 719dc50 Compare February 8, 2024 21:32
@FFHener FFHener requested a review from Noki February 8, 2024 21:35
Copy link
Member

@Noki Noki left a comment

Choose a reason for hiding this comment

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

looks good to me

@Noki Noki force-pushed the testbuild-wikiupdater branch from 719dc50 to 69617b5 Compare February 10, 2024 06:01
@Noki Noki merged commit 1fba288 into master Feb 10, 2024
6 checks passed
@Noki Noki deleted the testbuild-wikiupdater branch February 10, 2024 06:06
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