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 GitHub workflows to use new content tree (Issue 2418 task 10) #2464

Merged
merged 1 commit into from
Sep 7, 2022

Conversation

alflennik
Copy link
Contributor

See #2418 for more information about the task. Note that the build will fail until the changes on the move-examples-6 branch (#2441) are approved and merged.

@mcking65 mcking65 changed the title Implement 2418 task 10 Update GitHub workflows to use new content tree (Issue 2418 task 10) Sep 5, 2022
Copy link
Contributor

@mcking65 mcking65 left a comment

Choose a reason for hiding this comment

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

LGTM ... although this YML is definitely not something I fully understand.

Would be really nice if @nschonni were to also take a look.

Copy link
Contributor

@nschonni nschonni left a comment

Choose a reason for hiding this comment

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

Taking a quick look at the branch structure, it mostly makes sense, except that the aria-practices-DeletedSectionsArchive.html still exists (but maybe is supposed to be deleted in a different PR) and the old coding-templates I believe are now content-templates folder that needs to be ignored

@@ -6,9 +6,8 @@ on:
- ".github/workflows/regression.yml"
- "package*.json"
- "test/**"
- "examples/**"
- "!examples/landmarks/**"
- "!examples/coding-template/**"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this still exists as content-templates now

@mcking65
Copy link
Contributor

mcking65 commented Sep 6, 2022

I just removed aria-practices-DeletedSectionsArchive.html from the move-examples branch.

@alflennik
Copy link
Contributor Author

Thanks for the approval @mcking65 and the review @nschonni! Your comment is appreciated nschonni, it's definitely confusing to include references to changes that are actually occurring in other branches. But it sounds like Matt mitigated the issue by removing a file in the base branch.

@alflennik alflennik merged commit 22a3086 into move-examples Sep 7, 2022
@alflennik alflennik deleted the move-examples-10 branch September 7, 2022 18:04
@nschonni
Copy link
Contributor

nschonni commented Sep 7, 2022

Yeah, I delete the comment about the Archive file, but I don't think the one about the content templates was addressed

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.

3 participants