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

update Node to 6.x LTS #1228

Merged
merged 2 commits into from
Apr 15, 2017
Merged

update Node to 6.x LTS #1228

merged 2 commits into from
Apr 15, 2017

Conversation

dunn
Copy link
Contributor

@dunn dunn commented Apr 8, 2017

The 4.x branch entered maintenance on 1 April.

@krainboltgreene
Copy link
Member

Great call.

@krainboltgreene krainboltgreene requested a review from Gargron April 8, 2017 05:12
@krainboltgreene krainboltgreene added breaking Incompatible with previous versions requires in depth labels Apr 8, 2017
@krainboltgreene
Copy link
Member

Need to make sure everything works with the breaking changes from 4 -> 6. I'm pretty sure it'll be fine.

@dunn
Copy link
Contributor Author

dunn commented Apr 8, 2017

Dunno why Travis' nvm is getting confused; that version-string works fine locally.

.nvmrc Outdated
@@ -1 +1 @@
6.7.0
lts/boron
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps it expects a number version? Travis version of nvm could be different from yours.. I have not really used nvm before either, so I am not sure.

.travis.yml Outdated
@@ -28,7 +26,7 @@ before_install:
- sudo apt-get -qq update
- sudo apt-get -qq install g++-4.8
install:
- nvm install $TRAVIS_NODE_VERSION
- nvm install
- npm install -g npm@3
Copy link
Member

Choose a reason for hiding this comment

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

If Node 6+ comes with npm >=3, this line can be removed

@Gargron Gargron added the deployment Related to runtime configuration, production setups label Apr 8, 2017
@dunn dunn force-pushed the node-version branch 2 times, most recently from ac94609 to fd8dc07 Compare April 8, 2017 14:03
@dunn
Copy link
Contributor Author

dunn commented Apr 8, 2017

travis-ci/travis-ci#7593

The 4.x branch [entered maintenance](https://github.com/nodejs/LTS#lts-schedule1) on 1 April.
@dunn
Copy link
Contributor Author

dunn commented Apr 8, 2017

The Travis people replied; with language: ruby they won't guarantee a modern nvm, but I forgot we can just do 6 in the .nvmrc instead of specifying the LTS name.

@Gargron Gargron merged commit 8482f67 into mastodon:master Apr 15, 2017
@dunn dunn deleted the node-version branch April 15, 2017 03:01
Nyoho referenced this pull request in Nyoho/mathtodon Apr 25, 2017
The 4.x branch [entered maintenance](https://github.com/nodejs/LTS#lts-schedule1) on 1 April.
abcang added a commit to pixiv/mastodon that referenced this pull request Feb 25, 2019
hannahwhy pushed a commit to hannahwhy/mastodon that referenced this pull request Oct 5, 2019
dariusk added a commit to friendcamp/mastodon that referenced this pull request Dec 2, 2022
…icles

Make the "media gallery" not show up on Articles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Incompatible with previous versions deployment Related to runtime configuration, production setups
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants