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

Update to 2.1.0 #243

Merged
merged 9 commits into from
Aug 27, 2021
Merged

Update to 2.1.0 #243

merged 9 commits into from
Aug 27, 2021

Conversation

lossyrob
Copy link
Member

@lossyrob lossyrob commented Aug 26, 2021

This PR updates stac-fastapi to 2.1.0 as a first step to the release process.

Docs are currently broken on master because of a missing dependency in stac_fastapi.api. Doc building is currently not tested for PRs, and this PR adds docs creation to CI to catch this in the future.

This PR will also add changelog entries, as well as a PR template that will encourage contributors to make their own changelog entries at the time of feature/fix development.

This will catch errors around doc build failures on PRs.

This generates docs through a containerized environment, as opposed to
the GitHub Action, which builds it's own environment. We may want to
consider utilizing the dockerized environment for the action as well
to reduce any sync issues
@lossyrob lossyrob changed the title Update to 0.2.1 Update to 2.1.0 Aug 26, 2021
This commit modifies the changelog to be more in line with changelogs
of other stac-utils projects, and fit more to the 'keep a changelog' pattern.
Copy link
Collaborator

@moradology moradology left a comment

Choose a reason for hiding this comment

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

A small typo; looks good otherwise


### Fixed

* Make collection title is optional in landing page links ([#198](https://github.com/stac-utils/stac-fastapi/pull/198))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra "is" here

Copy link
Member Author

Choose a reason for hiding this comment

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

Ack, the UI hadn't shown this before I saw the approval and merge. I'll make a PR but release with current master

@lossyrob lossyrob merged commit b29d171 into master Aug 27, 2021
@geospatial-jeff geospatial-jeff deleted the release/2.1 branch August 1, 2022 21:28
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