Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

chore(docs.angularjs.org): allow robots in partials/ but prevent inde… #16457

Merged
merged 1 commit into from
Feb 15, 2018

Conversation

Narretz
Copy link
Contributor

@Narretz Narretz commented Feb 14, 2018

…xing

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change?

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Fix/Feature: Docs have been added/updated
  • Fix/Feature: Tests have been added; existing tests pass

Other information:

The sitemap.xml might also prevent the indexing, as the partials are not
listed.

Related to angular#16432
Copy link
Member

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

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

LGTM
(Not sure what happened with Travis. The job was cancelled. Did you cancel it?)

@Narretz
Copy link
Contributor Author

Narretz commented Feb 15, 2018

Yeah I changed the commit message and pushed it twice again and cancelles these jobs. The initial one is green

@mgol
Copy link
Member

mgol commented Feb 15, 2018

@Narretz In such cases I wouldn’t cancel the last job as it’s then confusing during the review.

@Narretz Narretz merged commit 2953e47 into angular:v1.6.x Feb 15, 2018
@Narretz Narretz deleted the chore-docs-noindex branch February 15, 2018 10:54
@Narretz
Copy link
Contributor Author

Narretz commented Feb 15, 2018

@mgol You are right, I cancelled the wrong builds - sorry

Narretz added a commit that referenced this pull request Feb 15, 2018
The sitemap.xml might also prevent the indexing, as the partials are not
listed.

Related to #16432 
Closes #16457
Narretz added a commit to Narretz/angular.js that referenced this pull request Feb 23, 2018
The sitemap.xml might also prevent the indexing, as the partials are not
listed.

Related to angular#16432
Closes angular#16457

Closes angular#16446
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants