-
Notifications
You must be signed in to change notification settings - Fork 224
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
report: Add macbox7 GSoC report #614
Conversation
reports/macbox7.md
Outdated
blog: https://macbox7.github.io/Blog-GSOC18/ | ||
|
||
activity: | ||
- 0: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these need to be indented.
we dont have a bear for this ;-(
reports/macbox7.md
Outdated
in a python file. Special handling is also done to detect violations | ||
in doctests which other similar plugins fail to perform. | ||
8. Create a `BuiltinsBear` that is capable of detecting redefinition | ||
of python builtins in a file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
file -> module
reports/macbox7.md
Outdated
|
||
#### Future prospects of the projects | ||
|
||
With `PyFlakesASTBear` ready bear developers can use it to detect codesmells |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"ready" isnt clear. Is it ready now? If so, using "with .. ready" is redundant, at best.
Also migrating flake8 plugins to this new infrastructure should be easy tasks that people can now work on.
And more importantly, we had planned for this bear framework to be able to load & run flake8 ast plugins as-is, so porting wasnt necessary; that can be mentioned as a medium to large sized project which is now feasible.
261f4f1
to
1073c00
Compare
reports/macbox7.md
Outdated
- 1: | ||
- repo: coala-pyflakes | ||
- link: https://gitlab.com/coala/bears/coala-pyflakes/commit/9484d8af901153c490812d86809760b14c46d9af | ||
- details: PyFlakesASTBear.py: Add PyFlakesASTBear |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use quotes or :
will be treated as defining yaml variable
reports/macbox7.md
Outdated
- 0: | ||
- repo: coala-pyflakes | ||
- link: https://gitlab.com/coala/bears/coala-pyflakes/commit/684f1a958dd66fa781e7f03777ffa96b7e6a2ca7 | ||
- details: nitial commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably a typo
722f998
to
5e853c0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a report_year
variable
Pretty sure you aren't a 2017 student :p
5e853c0
to
d41fffc
Compare
Travis tests have failedHey @MacBox7, |
d41fffc
to
b385241
Compare
reports/macbox7.md
Outdated
email: [email protected] | ||
blog: https://macbox7.github.io/Blog-GSOC18/ | ||
activity: | ||
- 0: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See changes requested at #615
reports/macbox7.md
Outdated
checker.py: Handle UnboundLocal error for builtins | ||
- 52: | ||
- repo: pyflakes | ||
- link: https://github.com/PyCQA/pyflakes/pull/329 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can get this merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merged
Also https://gitlab.com/coala/PyPrint/graphs/master and https://gitlab.com/coala/mobans/graphs/master and https://gitlab.com/coala/coala-utils/graphs/master , and maybe others on gitlab ;) |
reports/macbox7.md
Outdated
report_year: 2018 | ||
project: Integrate pyflakes-enhanced AST into coala | ||
project_link: https://summerofcode.withgoogle.com/projects/#5549789140221952 | ||
tarball: https://rawgit.com/MacBox7/GSoC-Report/master/project.tar.gz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to add another pyflakes patch into this
b385241
to
1e7e89d
Compare
reports/macbox7.md
Outdated
@@ -0,0 +1,489 @@ | |||
--- | |||
layout: report | |||
permalink: /GSoC/2018/StatusReport/macbox7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this and layout:
add date, categories, etc, as shown at https://github.com/coala/projects/pull/633/files#diff-f1164fdbd5814b6d270a59c0f9e3939e
1e7e89d
to
6067e0a
Compare
reports/macbox7.md
Outdated
[Adhika Setya Pramudita](https://github.com/adhikasp), | ||
[Steven Myint](https://github.com/myint) | ||
phase: | ||
- Bonding: https://gitlab.com/coala/GSoC/GSoC-2018/milestones/8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yamllint.
Also needs a rebase |
a66d9f8
to
5c85a2a
Compare
Still not rebased, and still not fixed #614 (comment) |
5c85a2a
to
3215087
Compare
[Adhika Setya Pramudita](https://github.com/adhikasp), | ||
[Steven Myint](https://github.com/myint) | ||
phase: | ||
- Bonding: https://gitlab.com/coala/GSoC/GSoC-2018/milestones/8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yamllint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No error on this line. The only error I am getting is =>
[ 473] The·challenge·of·the·project·was·to·implement·the·`PyFlakesASTBear`·in
**** YAMLLintBear [Section: cli | Severity: MAJOR] ****
! ! syntax error: expected '<document start>', but found '<scalar>'
[ ] *0. Do (N)othing
[ ] 1. (O)pen file
[ ] 2. Add (I)gnore comment
[ ] Enter number (Ctrl-D to exit):
@gitmate-bot rebase |
Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently |
3215087
to
e8b940b
Compare
Automated rebase with GitMate.io was successful! 🎉 |
@gitmate-bot ff |
Hey! I'm GitMate.io! This pull request is being fastforwarded automatically. Please DO NOT push while fastforward is in progress or your changes would be lost permanently |
Automated fastforward with GitMate.io was successful! 🎉 |
https://deploy-preview-614--coala-projects.netlify.com/gsoc/2018/statusreport/macbox7