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

Use absolute paths in Babel's CONTRIBUTING.md #5431

Merged
merged 2 commits into from
Mar 10, 2017
Merged

Conversation

aaronang
Copy link
Member

@aaronang aaronang commented Mar 8, 2017

Q A
Patch: Bug Fix? no
Major: Breaking Change? no
Minor: New Feature? no
Deprecations? no
Spec Compliancy? no
Tests Added/Pass? no
Fixed Tickets
License MIT
Doc PR yes
Dependency Changes no

Changing relative paths to absolute paths such that babel/website#1192 LinkCheck will pass. This also ensures that links will work on the Babel website.

@existentialism existentialism added the PR: Docs 📝 A type of pull request used for our changelog categories label Mar 8, 2017
@existentialism
Copy link
Member

Unrelated to this PR, but we should also fix the heading hierarchy (the headings under developing should be ### instead of ####)

This would result in a cleaner nav on the website:
image

vs.

image

CONTRIBUTING.md Outdated
@@ -3,15 +3,15 @@
----

<p align="center">
Copy link
Member

@existentialism existentialism Mar 8, 2017

Choose a reason for hiding this comment

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

Two things:

  • Let's add "toc" class to this, so we can hide these on the website, no sense in having TOC links here AND on the side.

  • Let's revert back to the anchor links, with these changes:

image

@existentialism existentialism dismissed their stale review March 10, 2017 14:49

pushed manually

@existentialism existentialism merged commit 911b71f into 7.0 Mar 10, 2017
@existentialism existentialism deleted the doc-relative-links branch March 10, 2017 14:49
@codecov
Copy link

codecov bot commented Mar 10, 2017

Codecov Report

Merging #5431 into 7.0 will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              7.0    #5431      +/-   ##
==========================================
- Coverage   85.43%   85.39%   -0.05%     
==========================================
  Files         203      203              
  Lines        9532     9533       +1     
  Branches     2702     2703       +1     
==========================================
- Hits         8144     8141       -3     
- Misses        901      905       +4     
  Partials      487      487
Impacted Files Coverage Δ
packages/babel-helper-call-delegate/src/index.js 64% <0%> (-4%)
packages/babel-traverse/src/path/modification.js 73.78% <0%> (-0.98%)
packages/babel-traverse/src/path/context.js 85.34% <0%> (-0.87%)
...bel-plugin-transform-es2015-classes/src/vanilla.js 89.74% <0%> (-0.86%)
packages/babel-generator/src/node/parentheses.js 97.02% <0%> (+0.02%)
packages/babel-traverse/src/visitors.js 86.66% <0%> (+0.95%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 02473a7...63b053e. Read the comment docs.

hulkish added a commit to hulkish/babel that referenced this pull request May 2, 2017
* 'master' of github.com:hulkish/babel: (37 commits)
  Increase the code coverage for traverse evaluation (babel#5363)
  Improve options documentation for `babel-plugin-transform-runtime` babel#5401
  [doc] Fix: comments in usage w/ options babel#5400
  document cache option for babel-register babel#5440
  Update coffescript/register reference link address babel#5475
  Update babel-generator's README babel#5517
  Improve example of babel-plugin-transform-es2015-arrow-functions babel#5573
  Remove incorrect docs. babel#5580
  Update transform-es2015-modules-commonjs doc babel#5588
  Fix replacing function declaration in export default (fixes babel#4468) (babel#5456)
  Fix a typo: occurences -> occurrences (babel#5575)
  Fix changelog [skip ci]
  Treat "await" as an invalid identifier (babel#4954)
  Removes unused lodash dependency from babel-helper-builder-react-jsx (babel#5486)
  docs: [skip ci] update documentation
  v6.24.0
  changelog for 6.24.0 [skip ci] (babel#5452)
  Keep parentheses for logical expression when in await expression (fix babel#5428) (babel#5433) (babel#5453)
  Use absolute paths in Babel's CONTRIBUTING.md (babel#5431)
  Fixed broken links in README.md [skip ci] (babel#5449)
  ...
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 6, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Docs 📝 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants