Skip to content

Commit d343a1d

Browse files
authored
Merge pull request #1321 from benjaoming/no-bumpversion-package-lock.json
Don't run bumpversion on package-lock.json
2 parents 487d517 + 47a2582 commit d343a1d

File tree

3 files changed

+62
-6
lines changed

3 files changed

+62
-6
lines changed

docs/contributing.rst

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,11 @@ To release a new version of the theme, core team will take the following steps:
184184
#. Update the changelog (``docs/changelog.rst``) with the version information.
185185
#. Run ``python setup.py update_translations`` to compile new translation files
186186
and update Transifex.
187-
#. Run ``python setup.py build_assets`` to rebuild all the theme assets and the Python
188-
package.
187+
#. Run ``npm install && npm run build`` to rebuild all the theme assets,
188+
notice that ``package-lock.json`` will be updated with a new package
189+
version. But take care that it isn't also bumping versions of dependencies
190+
in ways that are risky to the release. *If* ``package-lock.json`` changes, you
191+
need to commit it to git and tag it together with your release.
189192
#. Commit these changes.
190193
#. Tag the release in git: ``git tag $NEW_VERSION``.
191194
#. Push the tag to GitHub: ``git push --tags origin``.

setup.cfg

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ dev =
4343
transifex-client
4444
sphinxcontrib-httpdomain
4545
bump2version
46+
wheel
4647

4748
[options.entry_points]
4849
sphinx.html_themes =
@@ -107,10 +108,6 @@ first_value = 1
107108
search = "version": "{current_version}",
108109
replace = "version": "{new_version}",
109110

110-
[bumpversion:file:package-lock.json]
111-
search = "version": "{current_version}",
112-
replace = "version": "{new_version}",
113-
114111
[bumpversion:file:docs/changelog.rst]
115112
search = .. |development_version| replace:: {current_version}
116113
replace = .. |development_version| replace:: {new_version}

setup.py

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,4 +93,60 @@ def run(self):
9393
'build_assets': WebpackBuildCommand,
9494
'watch': WebpackDevelopCommand,
9595
},
96+
zip_safe=False,
97+
packages=['sphinx_rtd_theme'],
98+
package_data={'sphinx_rtd_theme': [
99+
'theme.conf',
100+
'*.html',
101+
'static/css/*.css',
102+
'static/css/fonts/*.*',
103+
'static/js/*.js',
104+
]},
105+
include_package_data=True,
106+
# See http://www.sphinx-doc.org/en/stable/theming.html#distribute-your-theme-as-a-python-package
107+
entry_points = {
108+
'sphinx.html_themes': [
109+
'sphinx_rtd_theme = sphinx_rtd_theme',
110+
]
111+
},
112+
python_requires='>=2.7,!=3.0.*,!=3.1.*,!=3.2.*,!=3.3.*',
113+
install_requires=[
114+
'sphinx>=1.6',
115+
'docutils<0.18',
116+
'Jinja2<3.1',
117+
],
118+
tests_require=[
119+
'pytest',
120+
],
121+
extras_require={
122+
'dev': [
123+
'transifex-client',
124+
'sphinxcontrib-httpdomain',
125+
'bump2version',
126+
'wheel',
127+
],
128+
},
129+
classifiers=[
130+
'Framework :: Sphinx',
131+
'Framework :: Sphinx :: Theme',
132+
'Development Status :: 5 - Production/Stable',
133+
'License :: OSI Approved :: MIT License',
134+
'Environment :: Console',
135+
'Environment :: Web Environment',
136+
'Intended Audience :: Developers',
137+
'Programming Language :: Python :: 2.7',
138+
'Programming Language :: Python :: 3',
139+
'Programming Language :: Python :: 3.6',
140+
'Programming Language :: Python :: 3.7',
141+
'Programming Language :: Python :: 3.8',
142+
'Programming Language :: Python :: 3.9',
143+
'Operating System :: OS Independent',
144+
'Topic :: Documentation',
145+
'Topic :: Software Development :: Documentation',
146+
],
147+
project_urls={
148+
'Homepage': 'https://sphinx-rtd-theme.readthedocs.io/',
149+
'Source Code': 'https://github.com/readthedocs/sphinx_rtd_theme',
150+
'Issue Tracker': 'https://github.com/readthedocs/sphinx_rtd_theme/issues',
151+
},
96152
)

0 commit comments

Comments
 (0)