Skip to content

Set isDevelopingAddon to false #111

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

Merged

Conversation

toddjordan
Copy link
Contributor

Publishing the addon with isDevelopingAddon to true causes undesirable things such as an app's linting evaluating styleguide addon files (which is happening with ember-api-docs right now).

@toddjordan toddjordan requested a review from MelSumner October 26, 2018 04:03
index.js Outdated
@@ -7,7 +7,7 @@ const nodeSass = require('node-sass');
module.exports = {
name: require('./package').name,
isDevelopingAddon() {
return true;
return false;
Copy link
Member

Choose a reason for hiding this comment

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

since this is the default may be either delete this method or block comment it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah I can do that

@toddjordan toddjordan force-pushed the set-isdeveloping-back-false branch from b407a0d to 69208f8 Compare October 26, 2018 04:12
@toddjordan
Copy link
Contributor Author

Will want to publish once merged because this is causing implementers to get template lint errors

@sivakumar-kailasam sivakumar-kailasam merged commit e29bf47 into ember-learn:master Oct 26, 2018
@sivakumar-kailasam
Copy link
Member

@toddjordan released a new version.

@toddjordan
Copy link
Contributor Author

Thanks!

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.

2 participants