Skip to content
This repository has been archived by the owner on Jan 6, 2023. It is now read-only.

README update #493

Merged
merged 3 commits into from
Sep 26, 2018
Merged

README update #493

merged 3 commits into from
Sep 26, 2018

Conversation

jf990
Copy link
Contributor

@jf990 jf990 commented Sep 24, 2018

  • Removed URLs with /en/
  • Use https
  • Corrects a few markdown lint errors
  • changed Copyright to 2018
  • Rebased to v.next

README.md Outdated
@@ -42,7 +44,8 @@ Find a bug or want to request a new feature? Please let us know by submitting a
Esri welcomes contributions from anyone and everyone. Please see our [guidelines for contributing](https://github.com/esri/contributing).

## Licensing
Copyright 2013 Esri

Copyright 2018 Esri
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it correct to change this to the current year? My understanding is that the year to include in a copyright notice is the year of first publication. Should this be Copyright 2013-2018 Esri?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was most certain it is the most recent date, but I will find out

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I got clarification on the copyright policy and updated it here http://devinfo.esri.com/sites/DeveloperCentral/github/SitePages/Policies.aspx

Thanks for pointing this out.

Copy link
Contributor

Choose a reason for hiding this comment

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

You're welcome. Thank you for looking into it.

@philium philium merged commit 01ce86d into Esri:v.next Sep 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants