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

Change default scales to 1.0 #5507

Closed
wants to merge 10 commits into from
Closed

Conversation

toslunar
Copy link
Member

Default scales should be 1.0.

Just in case, I added a config option to make them legacy.

@toslunar toslunar added the no-compat Change that disrupts backward compatibility. label Oct 17, 2018
@toslunar
Copy link
Member Author

The tests are too weak. Merge #5511 first.

@kmaehashi kmaehashi added the cat:enhancement Implementation that does not break interfaces. label Oct 19, 2018
@kmaehashi kmaehashi self-assigned this Oct 30, 2018
@kmaehashi
Copy link
Member

We should discuss if it is ok to introduce this change immediately. Raising a warning when scale is not specified is one idea.

@toslunar
Copy link
Member Author

Added warnings.

@stale
Copy link

stale bot commented Apr 25, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 30 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Not updated for a longer period of time. label Apr 25, 2019
@stale stale bot removed the stale Not updated for a longer period of time. label Apr 25, 2019
@stale
Copy link

stale bot commented Aug 8, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 30 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Not updated for a longer period of time. label Aug 8, 2019
@stale
Copy link

stale bot commented Sep 7, 2019

This issue is closed as announced. Feel free to re-open it if needed.

@stale stale bot closed this Sep 7, 2019
@kmaehashi kmaehashi added this to the Closed issues and PRs milestone Sep 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat:enhancement Implementation that does not break interfaces. no-compat Change that disrupts backward compatibility. stale Not updated for a longer period of time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants