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 gitignore #1671

Merged
merged 1 commit into from
Nov 20, 2024
Merged

add gitignore #1671

merged 1 commit into from
Nov 20, 2024

Conversation

SunJiFengPlus
Copy link
Contributor

@SunJiFengPlus SunJiFengPlus commented Nov 15, 2024

What type of PR is this?

enhance

What this PR does / why we need it?

keep the repository clean by excluding temporary files.

Which issue(s) this PR fixes?

Fixes #1668

Does this PR introduce a user-facing change?

No

Checklist

  • Make sure there is a GitHub_issue related with this PR before you start working on it.
  • Make sure you have squashed your change to one single commit.
  • GitHub Actions works fine in this PR.

Copy link

codecov bot commented Nov 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ Complexity Δ
unittests 43.92% <ø> (-0.04%) 181.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 18 files with indirect coverage changes


🚨 Try these New Features:

lilai23
lilai23 previously approved these changes Nov 19, 2024
daizhenyu
daizhenyu previously approved these changes Nov 19, 2024
provenceee
provenceee previously approved these changes Nov 20, 2024
@lilai23
Copy link
Collaborator

lilai23 commented Nov 20, 2024

check line lint action

@SunJiFengPlus
Copy link
Contributor Author

SunJiFengPlus commented Nov 20, 2024

What are the rules for checking line lint? I checked that my line sperator is LF(\n), do I need to end with a blank line?

Signed-off-by: sunjifeng <[email protected]>
@Sherlockhan Sherlockhan merged commit bc4649f into sermant-io:develop Nov 20, 2024
6 checks passed
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.

add .gitignore
6 participants