Skip to content

Conversation

@katrinleinweber
Copy link
Contributor

What changes were proposed in this pull request?

The DOI foundation recommends this new resolver. Accordingly, this PR reseds all static DOI links ;-)

How was this patch tested?

It wasn't, since it seems as safe as a "typo fix".

In case any of the files is included from other projects, and should be updated there, please let me know.

@katrinleinweber katrinleinweber changed the title Hyperlink DOIs to preferred resolver [MINOR] Update all DOI links to preferred resolver Nov 24, 2018
@rxin
Copy link
Contributor

rxin commented Nov 24, 2018

Jenkins, test this please.

@HyukjinKwon
Copy link
Member

ok to test

@SparkQA
Copy link

SparkQA commented Nov 25, 2018

Test build #99237 has finished for PR 23129 at commit 2e14ad8.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Nov 25, 2018

Merged to master

@asfgit asfgit closed this in c5daccb Nov 25, 2018
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
## What changes were proposed in this pull request?

The DOI foundation recommends [this new resolver](https://www.doi.org/doi_handbook/3_Resolution.html#3.8). Accordingly, this PR re`sed`s all static DOI links ;-)

## How was this patch tested?

It wasn't, since it seems as safe as a "[typo fix](https://spark.apache.org/contributing.html)".

In case any of the files is included from other projects, and should be updated there, please let me know.

Closes apache#23129 from katrinleinweber/resolve-DOIs-securely.

Authored-by: Katrin Leinweber <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
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.

5 participants