Skip to content

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Apr 19, 2017

/cc @jonparrott

@lukesneeringer You mentioned that you'd prefer a standalone session for this. I am unclear why you prefer this. To me, checking that the README isn't invalid in setup.py is linting.

@dhermes dhermes added api: datastore Issues related to the Datastore API. hygiene labels Apr 19, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 19, 2017
@lukesneeringer lukesneeringer added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Apr 19, 2017
@lukesneeringer
Copy link
Contributor

My rationale is:

  • These are separate and orthogonal checks. If one fails and one succeeds, I want to know that.
  • Different virtualenvs ensures atomicity.

@dhermes dhermes force-pushed the fix-2446-datastore branch from 51b7b03 to ec98ae7 Compare April 19, 2017 22:25
@dhermes
Copy link
Contributor Author

dhermes commented Apr 19, 2017

OK @lukesneeringer PTAL

@lukesneeringer lukesneeringer merged commit 5d99380 into googleapis:master Apr 20, 2017
@lukesneeringer lukesneeringer removed the priority: p2 Moderately-important priority. Fix may not be included in next release. label Apr 20, 2017
@dhermes dhermes deleted the fix-2446-datastore branch April 20, 2017 16:34
dhermes added a commit to dhermes/google-cloud-python that referenced this pull request Apr 20, 2017
dhermes added a commit to dhermes/google-cloud-python that referenced this pull request Apr 20, 2017
dhermes added a commit that referenced this pull request Apr 20, 2017
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
…s#3318)

* Adding check that **all** setup.py README's are valid RST.

Follow up to googleapis#3316. Fixes googleapis#2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
atulep pushed a commit that referenced this pull request Apr 3, 2023
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
atulep pushed a commit that referenced this pull request Apr 18, 2023
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
parthea pushed a commit that referenced this pull request Jun 4, 2023
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
parthea pushed a commit that referenced this pull request Jul 6, 2023
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
parthea pushed a commit that referenced this pull request Oct 21, 2023
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
parthea pushed a commit that referenced this pull request Oct 21, 2023
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
parthea pushed a commit that referenced this pull request Oct 21, 2023
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
parthea pushed a commit that referenced this pull request Oct 22, 2023
* Adding check that **all** setup.py README's are valid RST.

Follow up to #3316. Fixes #2446.

* Fixing duplicate reference in Logging README.

* Fixing duplicate reference in Monitoring README.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: datastore Issues related to the Datastore API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants