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

Feature/GitHub action publish to staging #60

Merged
merged 42 commits into from
Mar 6, 2023

Conversation

rugeli
Copy link
Collaborator

@rugeli rugeli commented Mar 3, 2023

Problem

What is the problem this work solves, including
closes #49

Solution

What I/we did to solve this problem

  • Added repo secrets in github for db access
  • Created a yaml file in github/workflows that will be manually triggered and run process-dataset
  • Input includes the path to the dataset and a boolean that indicates whether to skip the file info upload

with @meganrm

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update (we can update the documentation when the release dataset to production db is done)

Steps to Verify:

  1. Go to Actions and choose upload dataset to staging DB in workflows
  2. Click on Run Workflow and select feature/github-action-publish-to-staging branch
  3. Then enter the path to the new dataset
  4. Check the box if we want to skip the file-info upload
  5. Run workflow

@rugeli rugeli requested a review from toloudis as a code owner March 3, 2023 21:10
@rugeli rugeli requested a review from meganrm March 3, 2023 21:10
@rugeli rugeli merged commit 05293e3 into main Mar 6, 2023
@rugeli rugeli deleted the feature/github-action-publish-to-staging branch March 17, 2023 18:40
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.

data creators should be able to publish to staging
3 participants