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

Add checks for file mode changes and absolute imports #2398

Merged
merged 3 commits into from
Aug 28, 2021

Conversation

wjsi
Copy link
Member

@wjsi wjsi commented Aug 28, 2021

What do these changes do?

Add checks below:

  1. File mode changes. This forces commiters to ignore fileMode in Windows.
  2. Absolute imports in non-test source code.

Also add a .gitattributes to avoid CRLF line endings in Windows.

Related issue number

@wjsi wjsi added type: tests to be backported Indicate that the PR need to be backported to stable branch labels Aug 28, 2021
@wjsi wjsi added this to the v0.8.0a3 milestone Aug 28, 2021
@wjsi wjsi force-pushed the tests/attr_file branch 3 times, most recently from 13c3f6f to 5d3abb8 Compare August 28, 2021 04:25
Copy link
Collaborator

@qinxuye qinxuye left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@hekaisheng hekaisheng left a comment

Choose a reason for hiding this comment

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

LGTM

@hekaisheng hekaisheng merged commit 1e4fa65 into mars-project:master Aug 28, 2021
@wjsi wjsi deleted the tests/attr_file branch August 28, 2021 11:02
wjsi added a commit to wjsi/mars that referenced this pull request Sep 1, 2021
@qinxuye qinxuye added backported already PR has been backported and removed to be backported Indicate that the PR need to be backported to stable branch labels Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported already PR has been backported type: tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants