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

Follow-up grunt for #472 #476

Merged
merged 2 commits into from Oct 15, 2017
Merged

Conversation

petermd
Copy link
Contributor

@petermd petermd commented Oct 13, 2017

Ran grunt build to re-generate the static js as per #472.

Please note: I did not

  • update any version number
  • include the re-generated css as there's a non-obvious diff (formatting of generated css?) in the css and css.map files but since the change has no CSS in it, I figured it was best to omit it

Let me know if thats ok

@Blendify
Copy link
Member

No, we need the css too from other PRs

@petermd
Copy link
Contributor Author

petermd commented Oct 14, 2017

K, figured out my issue, had sass 3.5.2 installed which changes the default precision hence the large diffs. Triedbundle exec grunt build instead but the Gemfile.lock is pretty old so not sure there's a simple way to get a repeatable env? (if you have newer versions of the dependencies)

In any case, re-installing the required gems and re-running has picked up the CSS changes to theme.css - let me know if you need anything else.

@Blendify Blendify merged commit 9d704f2 into readthedocs:master Oct 15, 2017
@Blendify
Copy link
Member

Thanks

@mattklein123
Copy link

Thank you @petermd !!! FYI envoyproxy/envoy#1860 💯

@petermd petermd deleted the avoid-auto-reset-grunt branch October 19, 2017 22:20
travismiller pushed a commit to broco1974/couscous-template-readthedocs that referenced this pull request Dec 4, 2017
* commit '6c2302472fb2c61eec9b4be3a2515117fffbfac3': (253 commits)
  Add pygments_style to theme.conf
  Document new theme option
  Add option to layout.html
  Update theme.conf
  Apply clearfix on breadcrumbs
  Follow-up grunt for readthedocs#472 (readthedocs#476)
  Only change current if a matching link is found (readthedocs#472)
  Add a couple of badge icons to readme (readthedocs#471)
  Add Sphinx Framework for PyPi (readthedocs#470)
  Fix error in setup entry points (readthedocs#466)
  Added `.admonition` to the CSS (readthedocs#462)
  Use explicit UTF8 encoding (readthedocs#452)
  Add missing pygments css link
  Convert underscores to dashes.
  Reflect language in lang attribute.
  Replace master with v0.2.5 in README.rst
  Add missing dependency in dev setup: sphinxcontrib-httpdomain (readthedocs#431)
  Finalized readthedocs#418 (readthedocs#433)
  Added a link to the Sphinx documentation about setuptools entry points
  Added setuptools entry point for Sphinx
  ...
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.

None yet

3 participants