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.

Sign up for GitHub

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

Jump to bottom

feat: add automated publishing workflow with no-ops #185

Merged
merged 7 commits into from
Nov 20, 2024
Merged

feat: add automated publishing workflow with no-ops #185

merged 7 commits into from
Nov 20, 2024

Conversation

botanical
Copy link
Member

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 19, 2024 •
edited
Loading

Workflow Status

Starting workflow... View action run

Collection Publication Status

  • hls-swir-falsecolor-composite-TEST: Successfully published ✅

@botanical botanical requested a review from anayeaye November 19, 2024 19:03
@botanical botanical marked this pull request as ready for review November 19, 2024 19:03
@botanical botanical requested a review from smohiudd as a code owner November 19, 2024 19:03
anayeaye
anayeaye reviewed Nov 19, 2024
View reviewed changes
.github/workflows/pr.yml Outdated Show resolved Hide resolved
.github/workflows/pr.yml Outdated Show resolved Hide resolved
.github/workflows/pr.yml Outdated Show resolved Hide resolved
.github/workflows/pr.yml Outdated Show resolved Hide resolved
.github/workflows/pr.yml Outdated Show resolved Hide resolved
botanical and others added 5 commits November 19, 2024 15:22
anayeaye
anayeaye approved these changes Nov 20, 2024
View reviewed changes
Copy link
Contributor

@anayeaye anayeaye left a comment

Choose a reason for hiding this comment

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

I'm OK with persisting the trigger on the testing/dataset-config folder a little longer OR with adding the event to the staging/dataset-config as @smohiudd suggested.

@smohiudd
Copy link
Contributor

Based on our discussions on slack, I'm ok if we keep these actions only on the testing directory for now.

@botanical botanical merged commit 4d38db5 into main Nov 20, 2024
6 checks passed
@botanical botanical deleted the issue-180/automated-staging-publishing branch November 20, 2024 22:30
@botanical botanical mentioned this pull request Nov 20, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smohiudd smohiudd smohiudd approved these changes

@anayeaye anayeaye anayeaye approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@botanical @smohiudd @anayeaye