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

docs: minor readme update #470

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

abarciauskas-bgse
Copy link
Contributor

Some minor setup changes I needed to run veda-backend locally

anayeaye and others added 2 commits February 18, 2025 07:10
### Changed

- Python 3.9 to 3.11 upgrade
- Pydantic>2 and associated package upgrades

### Breaking
- Pydantic upgrades require migrating pgstac version to 0.8.5 

### Added

- New custom surface temperature colormap (`st_cmap`) added to raster
api
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR makes minor updates to the README.md to improve local setup instructions for veda-backend.

  • Updates the installation commands for node version management.
  • Adds clarifying comments and commands for environment setup.

Co-authored-by: Copilot <[email protected]>
@abarciauskas-bgse
Copy link
Contributor Author

I will address the linting errors 👍🏽

@abarciauskas-bgse abarciauskas-bgse changed the title Minor readme update docs: minor readme update Mar 28, 2025
brew install jq
nvm install 20 # .github/workflows/pr.yml users node version 20
Copy link
Collaborator

Choose a reason for hiding this comment

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

on line 100 we run nvm use --lts, can you update that, too?

Copy link
Collaborator

@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.

Approve with one one small request to update nvm use --lts.

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.

2 participants