Skip to content

chore: update readme#39

Merged
emmalu merged 2 commits intomainfrom
chore/update-readme
Apr 9, 2026
Merged

chore: update readme#39
emmalu merged 2 commits intomainfrom
chore/update-readme

Conversation

@emmalu
Copy link
Copy Markdown
Contributor

@emmalu emmalu commented Apr 9, 2026

Updating the README with note about the routine csda version updates.

@emmalu emmalu requested review from a team and camillecroft April 9, 2026 21:10
Comment thread README.md

To see this in action, check out https://github.com/gadomski/csdap-demo-repo.
To see this in action, check out https://github.com/NASA-IMPACT/csdap-frontend.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Would it be helpful to point to the frontend release-please workflow?
https://github.com/NASA-IMPACT/csdap-frontend/blob/main/.github/workflows/release-please.yml

Comment thread README.md
token: ${{ secrets.GITHUB_TOKEN }}
release-type: simple
release-as: ${{ steps.csda-version.outputs.version }}
```
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sure if we want to update this too?

  steps:
    - name: CSDA Version
      id: csda-version
      uses: NASA-IMPACT/csda-version@<hash-version>
    - uses: googleapis/release-please-action@<hash-version>
      with:
        token: ${{ secrets.GITHUB_TOKEN }}
        release-type: simple
        release-as: ${{ steps.csda-version.outputs.version }}

gadomski
gadomski previously approved these changes Apr 9, 2026
Copy link
Copy Markdown
Contributor

@camillecroft camillecroft left a comment

Choose a reason for hiding this comment

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

Thank you for updating!

@emmalu emmalu merged commit 25916e4 into main Apr 9, 2026
1 check passed
@emmalu emmalu deleted the chore/update-readme branch April 9, 2026 23:05
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