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

Development: Clean up .github/workflows/docs.yml variables #7477

Merged
merged 9 commits into from
Nov 27, 2023

Conversation

ignacio-gn
Copy link
Member

Checklist

General

Motivation and Context

This PR is related to Development: Migrate documentation to docs.artemis.cit.tum.de #7298, and re-organizes the documentation deployment workflow in order to keep a clean state of GitHub repository secrets.

Description

Secrets of .github/workflows/docs.yml were converted to (environment) variables, in order to have a cleaner state of the repository secrets. Once this PR is merged, the unused secrets have to be removed.

Permissions set for the .github/workflows/docs.yml workflow were also cleaned up, as they were only relevant to the (now stale) GitHub Pages deployment.

Triggers set for the .github/workflows/docs.yml workflow were modified so that the documentation is not redeployed using the code of new PRs.

Steps for Testing

The workflow has been tested on a separate repository using an identical environment. This PR does not modify the functionality of Artemis.

@ignacio-gn ignacio-gn requested a review from Hialus October 30, 2023 17:49
@ignacio-gn ignacio-gn requested a review from a team as a code owner October 30, 2023 17:49
Copy link

github-actions bot commented Nov 9, 2023

There hasn't been any activity on this pull request recently. Therefore, this pull request has been automatically marked as stale and will be closed if no further activity occurs within seven days. Thank you for your contributions.

@github-actions github-actions bot added the stale label Nov 9, 2023
@github-actions github-actions bot removed the stale label Nov 20, 2023
Copy link

There hasn't been any activity on this pull request recently. Therefore, this pull request has been automatically marked as stale and will be closed if no further activity occurs within seven days. Thank you for your contributions.

@github-actions github-actions bot added the stale label Nov 27, 2023
Copy link
Member

@Hialus Hialus left a comment

Choose a reason for hiding this comment

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

Code looks good

@ignacio-gn ignacio-gn merged commit 20fd48e into ls1intum:develop Nov 27, 2023
2 checks passed
@krusche krusche added this to the 6.7.1 milestone Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants