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

report: Add macbox7 GSoC report #614

Merged
merged 1 commit into from
Aug 13, 2018

Conversation

ankitxjoshi
Copy link
Contributor

@ankitxjoshi ankitxjoshi commented Aug 9, 2018

@jayvdb
Copy link
Member

jayvdb commented Aug 9, 2018

blog: https://macbox7.github.io/Blog-GSOC18/

activity:
- 0:
Copy link
Member

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 ;-(

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.
Copy link
Member

Choose a reason for hiding this comment

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

file -> module


#### Future prospects of the projects

With `PyFlakesASTBear` ready bear developers can use it to detect codesmells
Copy link
Member

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.

- 1:
- repo: coala-pyflakes
- link: https://gitlab.com/coala/bears/coala-pyflakes/commit/9484d8af901153c490812d86809760b14c46d9af
- details: PyFlakesASTBear.py: Add PyFlakesASTBear
Copy link
Member

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

- 0:
- repo: coala-pyflakes
- link: https://gitlab.com/coala/bears/coala-pyflakes/commit/684f1a958dd66fa781e7f03777ffa96b7e6a2ca7
- details: nitial commit
Copy link
Member

@virresh virresh Aug 9, 2018

Choose a reason for hiding this comment

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

Probably a typo

@ankitxjoshi ankitxjoshi force-pushed the report-pyflakes branch 2 times, most recently from 722f998 to 5e853c0 Compare August 9, 2018 18:16
Copy link
Member

@virresh virresh left a 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

@ankitxjoshi ankitxjoshi changed the title reports: Add coala-pyflakes report report: Add virresh's GSoC report Aug 10, 2018
@ankitxjoshi ankitxjoshi changed the title report: Add virresh's GSoC report report: Add macbox7 GSoC report Aug 10, 2018
@TravisBuddy
Copy link

Travis tests have failed

Hey @MacBox7,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

email: [email protected]
blog: https://macbox7.github.io/Blog-GSOC18/
activity:
- 0:
Copy link
Member

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

checker.py: Handle UnboundLocal error for builtins
- 52:
- repo: pyflakes
- link: https://github.com/PyCQA/pyflakes/pull/329
Copy link
Member

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.

Copy link
Member

Choose a reason for hiding this comment

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

merged

@jayvdb
Copy link
Member

jayvdb commented Aug 11, 2018

@jayvdb
Copy link
Member

jayvdb commented Aug 11, 2018

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
Copy link
Member

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

@@ -0,0 +1,489 @@
---
layout: report
permalink: /GSoC/2018/StatusReport/macbox7
Copy link
Member

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

[Adhika Setya Pramudita](https://github.com/adhikasp),
[Steven Myint](https://github.com/myint)
phase:
- Bonding: https://gitlab.com/coala/GSoC/GSoC-2018/milestones/8
Copy link
Member

Choose a reason for hiding this comment

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

yamllint.

@jayvdb
Copy link
Member

jayvdb commented Aug 13, 2018

Also needs a rebase

@jayvdb
Copy link
Member

jayvdb commented Aug 13, 2018

Still not rebased, and still not fixed #614 (comment)

[Adhika Setya Pramudita](https://github.com/adhikasp),
[Steven Myint](https://github.com/myint)
phase:
- Bonding: https://gitlab.com/coala/GSoC/GSoC-2018/milestones/8
Copy link
Member

Choose a reason for hiding this comment

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

yamllint

Copy link
Contributor Author

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): 

@jayvdb
Copy link
Member

jayvdb commented Aug 13, 2018

@gitmate-bot rebase

@gitmate-bot
Copy link

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 ⚠️

@gitmate-bot
Copy link

Automated rebase with GitMate.io was successful! 🎉

@jayvdb
Copy link
Member

jayvdb commented Aug 13, 2018

@gitmate-bot ff

@gitmate-bot
Copy link

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 ⚠️

@gitmate-bot gitmate-bot merged commit e8b940b into coala:master Aug 13, 2018
@gitmate-bot
Copy link

Automated fastforward with GitMate.io was successful! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

5 participants