[go: nahoru, domu]

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

ignore .idea directory in dart analyzer and analyzer benchmark output in git #55235

Merged
merged 1 commit into from
Jul 30, 2020

Conversation

flar
Copy link
Contributor
@flar flar commented Apr 20, 2020

I recently ran dev/bots/analyze.dart in my repo to check for license violations and it bombed out by trying to analyze the files in my Android Studio .idea directory. That directory is in the .gitignore file, but analyze.dart doesn't use .gitignore to prune its files so I had to add a rule to the dart code to avoid this.

Also, analyze.dart outputs a benchmark json file when it is done with information on its performance. I added this file to the .gitignore to keep it out of git activities.

@fluttergithubbot fluttergithubbot added the team Infra upgrades, team productivity, code health, technical debt. See also team: labels. label Apr 20, 2020
@fluttergithubbot
Copy link
Contributor

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@flar
Copy link
Contributor Author
flar commented Apr 20, 2020

(Ignoring the boilerplate "needs test" bot...)

Copy link
Member
@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

@flar flar merged commit 210f768 into flutter:master Jul 30, 2020
@flar flar deleted the ignore-idea-dir-in-analyzer branch September 4, 2020 19:21
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team Infra upgrades, team productivity, code health, technical debt. See also team: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants