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

Formatting cleanup #711

Merged
merged 1 commit into from
Dec 4, 2022
Merged

Conversation

OMEGARAZER
Copy link
Contributor

Cleanup some formatting from switch to Black

Cleanup some formatting from switch to Black
@OMEGARAZER
Copy link
Contributor Author

fyi, the formatting check fails with this on my fork. Not sure if it's something you're familiar with or know why.

stdout: format_check run-test: commands[2] | mdl README.md docs/ -s .markdown_style.rb

stdout: /var/lib/gems/2.7.0/gems/mdl-0.12.0/lib/mdl/cli.rb:118:in `expand_path': couldn't find login name -- expanding `~' (ArgumentError)
	from /var/lib/gems/2.7.0/gems/mdl-0.12.0/lib/mdl/cli.rb:118:in `run'
	from /var/lib/gems/2.7.0/gems/mdl-0.12.0/lib/mdl.rb:16:in `run'
	from /var/lib/gems/2.7.0/gems/mdl-0.12.0/bin/mdl:10:in `<top (required)>'
	from /usr/local/bin/mdl:23:in `load'
	from /usr/local/bin/mdl:23:in `<main>'

stdout: ERROR: InvocationError for command /usr/local/bin/mdl README.md docs/ -s .markdown_style.rb (exited with code 1)

stdout: ___________________________________ summary ____________________________________
stdout: 
ERROR:   format_check: commands failed

@Serene-Arc
Copy link
Owner

I'm not sure what's causing that, does mdl work normally for you?

@Serene-Arc Serene-Arc merged commit 76b441c into Serene-Arc:development Dec 4, 2022
@OMEGARAZER
Copy link
Contributor Author

With Ruby and everything installed it does locally for me but not on my github branch. It doesn't stop the other tests from working. Just now get notifications for every push that it's failed when everything else passes.

@Serene-Arc
Copy link
Owner

That's weird because it works in mine and in Ali's, obviously since we were able to merge this PR with all checks green. Maybe open an issue and we can discuss it further. I know that the ruby version indicated in that stacktrace is older than the one I'm using on my PC.

@OMEGARAZER
Copy link
Contributor Author

Yeah, same, it's older than mine as well. No clue, oh well.

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.

2 participants