AstuteSource / chasten

:dizzy: Chasten Uses XML and XPATH to Check a Python Program's AST for Specified Patterns!
https://pypi.org/project/chasten/
GNU General Public License v2.0
7 stars 8 forks source link

feat: give analyze the ability to display browsable results #94

Closed Poiuy7312 closed 11 months ago

Poiuy7312 commented 11 months ago

closes #30

This gives analyze the sub commands --markdown-storage/-r and --display for storing the results in a markdown file and displaying it as a browsable frogmouth display when using --display

Ex

chasten analyze chasten --config $PWD/.chasten/ --debug-level ERROR --debug-dest CONSOLE --search-path . -r . --display

💫 chasten: Analyze the AST of Python Source Code
🔗 GitHub: https://github.com/gkapfham/chasten

✨ Configuration directory: C:\Users\Preston\CMPSC-203\ChastenProject\chasten\.chasten

✨ Validated C:\Users\Preston\CMPSC-203\ChastenProject\chasten\.chasten/config.yml? Yes
✨ Validated C:\Users\Preston\CMPSC-203\ChastenProject\chasten\.chasten/checks.yml? Yes

🐸 Frogmouth Information

 • Venv: C:\Users\Preston\AppData\Local\pypoetry\Cache\virtualenvs\chasten-Ti8Qy4ta-py3.11
 • Program: C:\Users\Preston\AppData\Local\pypoetry\Cache\virtualenvs\chasten-Ti8Qy4ta-py3.11/Scripts/frogmouth.exe

Display example:

image_2023-10-31_152614171

Poiuy7312 commented 11 months ago

I'm not sure if this is what was wanted but this is my attempt of giving something that stores the results in a readable form

gkapfham commented 11 months ago

Hi @Poiuy7312 can you give an example of what the command-line looks like? Can you also please give a screenshot that shows what output this produces?

gkapfham commented 11 months ago

Thanks @Poiuy7312 this is an awesome feature!

gkapfham commented 11 months ago

Hello @Poiuy7312 can you confirm that this is only for the analyze sub-command?

gkapfham commented 11 months ago

Also @tuduun, @laurennevill, and/or @jnormile can you review this and provide feedback?

Poiuy7312 commented 11 months ago

Yes this is only for analyze

On Wed, Nov 1, 2023, 16:44 Gregory M. Kapfhammer @.***> wrote:

Hello @Poiuy7312 https://github.com/Poiuy7312 can you confirm that this is only for the analyze sub-command?

— Reply to this email directly, view it on GitHub https://github.com/AstuteSource/chasten/pull/94#issuecomment-1789652256, or unsubscribe https://github.com/notifications/unsubscribe-auth/A2ZWPMBLUFSQ5VSBTYWWGWDYCKYDTAVCNFSM6AAAAAA6OEC3OGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGY2TEMRVGY . You are receiving this because you were mentioned.Message ID: @.***>

boulais01 commented 11 months ago

Something I just noticed as well - make sure you update to master as well

simojo commented 11 months ago

Just as a note, merging master into your branch still hasn't fixed the downgrading of poetry.lock.

boulais01 commented 11 months ago

@Poiuy7312 I was going to try and resolve the conflicts between the master main.py and the main.py here, but I'm not sure what some of the resolutions should be to maintain functionality. It's only three conflicts, so it shouldn't take long to resolve when you get the chance.