Skip to content

Summaries updates - to 'strongly recommended' and with a best practice#985

Merged
m-mohr merged 9 commits intodevfrom
summaries-updates
Feb 23, 2021
Merged

Summaries updates - to 'strongly recommended' and with a best practice#985
m-mohr merged 9 commits intodevfrom
summaries-updates

Conversation

@cholmes
Copy link
Copy Markdown
Contributor

@cholmes cholmes commented Feb 10, 2021 •
edited
Loading

Related Issue(s): #820 #965

Proposed Changes:

  1. Made summaries to be 'strongly recommended' and explained them a bit more
  2. added a best practices section on summaries.

PR Checklist:

@cholmes cholmes changed the title Summaries updates Summaries updates - to 'strongly recommended' and with a best practice Feb 10, 2021
@cholmes cholmes mentioned this pull request Feb 10, 2021
Closed
philvarner
philvarner reviewed Feb 10, 2021
View reviewed changes
philvarner
philvarner reviewed Feb 10, 2021
View reviewed changes
philvarner
philvarner reviewed Feb 10, 2021
View reviewed changes
philvarner
philvarner reviewed Feb 10, 2021
View reviewed changes
Copy link
Copy Markdown
Collaborator

@philvarner philvarner left a comment

Choose a reason for hiding this comment

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

few wordsmithing comments, looks good!

@cholmes
Copy link
Copy Markdown
Contributor Author

cholmes commented Feb 10, 2021

Wordsmith edits incorporated, thanks!

@jisantuc jisantuc mentioned this pull request Feb 10, 2021
Closed
matthewhanson
matthewhanson reviewed Feb 11, 2021
View reviewed changes
m-mohr
m-mohr reviewed Feb 11, 2021
View reviewed changes
m-mohr
m-mohr requested changes Feb 11, 2021
View reviewed changes
Copy link
Copy Markdown
Collaborator

@m-mohr m-mohr left a comment

Choose a reason for hiding this comment

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

One minor thing to add.

m-mohr
m-mohr reviewed Feb 11, 2021
View reviewed changes
m-mohr
m-mohr reviewed Feb 11, 2021
View reviewed changes
@cholmes @m-mohr
Update best-practices.md
6e51d33 
Co-authored-by: Matthias Mohr <[email protected]>
m-mohr
m-mohr approved these changes Feb 23, 2021
View reviewed changes
Copy link
Copy Markdown
Collaborator

@m-mohr m-mohr left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks, looks good now. I think we can do the stac-fields link later, once it's stable (see #1004).

@m-mohr m-mohr mentioned this pull request Feb 23, 2021
Closed
@m-mohr m-mohr merged commit d651e4a into dev Feb 23, 2021
@m-mohr m-mohr deleted the summaries-updates branch February 23, 2021 09:33
@m-mohr m-mohr mentioned this pull request Feb 23, 2021
Closed
@schwehr schwehr mentioned this pull request Apr 14, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@philvarner philvarner philvarner left review comments

@matthewhanson matthewhanson matthewhanson approved these changes

@m-mohr m-mohr m-mohr approved these changes

+1 more reviewer

@jbants jbants jbants approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@cholmes @philvarner @matthewhanson @m-mohr @jbants