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 extensions readme #1205

Merged
merged 2 commits into from
Jan 9, 2023
Merged

Update extensions readme #1205

merged 2 commits into from
Jan 9, 2023

Conversation

m-mohr
Copy link
Collaborator

@m-mohr m-mohr commented Dec 6, 2022

Related Issue(s): #1203

Proposed Changes:

  1. Remove the list of STAC extensions, only refer to stac-extensions.github.io for easier maintenance and updates
  2. A lot of changes and improvements in wording/text, cater for recent changes e.g. in how we handle and write STAC extensions (scope list)
  3. Better explain that you can basically extend every object
  4. Tried to remove a bit of duplication
  5. Moved extension maturity a level up as it is important for most chapters, not only for extending STAC

PR Checklist:

  • This PR is made against the dev branch (all proposed changes except releases should be against dev, not master).
  • This PR has no breaking changes.
  • I have added my changes to the CHANGELOG
    or a CHANGELOG entry is not required.
  • This PR affects the STAC API spec,
    and I have opened issue/PR #XXX to track the change.

@m-mohr m-mohr requested a review from cholmes December 6, 2022 11:43
Copy link
Contributor

@cholmes cholmes left a comment

Choose a reason for hiding this comment

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

looks good, thanks!

@m-mohr m-mohr added this to the 1.0.1 milestone Jan 5, 2023
extensions/README.md Outdated Show resolved Hide resolved
@m-mohr m-mohr merged commit a5c65d7 into dev Jan 9, 2023
@m-mohr m-mohr deleted the extensions-update branch January 9, 2023 17:07
@m-mohr m-mohr modified the milestones: 1.0.1, 1.1 May 2, 2023
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.

None yet

3 participants