-
Notifications
You must be signed in to change notification settings - Fork 51
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
Prepare a new release #42
Conversation
Perfect. I think jump to v1.0. |
@andrewharvey Hi, when v1.0 will be released? |
Any news on this? |
README.md
Outdated
var mapboxgl = require('mapbox-gl') | ||
var MapboxLanguage = require('@mapbox/mapbox-gl-language'); | ||
const mapboxgl = require('mapbox-gl') | ||
const MapboxLanguage = require('@mapbox/mapbox-gl-language'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: should we use the ESM syntax for these examples instead of require
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch on the old code. Fixed.
This PR prepares a new release of GL Language.
{name
).Do we want to jump to v1.0.0? Technically, since this is on major version 0, we're free to introduce breaking changes, but I think it makes sense to bump the major version here. It doesn't seem like the API is going to change radically so we can solidify it with the 1.0 release.
cc @andrewharvey