-
Notifications
You must be signed in to change notification settings - Fork 0
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
✨ Add PRE_COMMAND
to megalinter to install python dependencies before running linter
#81
Conversation
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.
🧪
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.
🌝🌖🚀LGTM🚀🌘🌚
🦙 MegaLinter status: ❌ ERROR
See detailed report in MegaLinter reports |
tests/views/test_views.py
Outdated
def dummy_python_change_to_test_linter(self): | ||
self.assertEqual(True, True) |
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.
I assume we don't need this anymore?
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.
Nice one! 🚀
* main: Restored removed logic for displaying message Updated description on test config Corrected tag name on checkbox Added new test config Selectable organisations added to both development and production configuration files Logic added to determine the organisations to display ✨ Add `PRE_COMMAND` to install python dependencies before running MegaLinter (#81)
👀 Purpose
♻️ What's changed
📝 Notes
pip install -r /github/workspace/requirements.txt
- essentially the working directory is the pylint virtual env, and/github/workspace
is the root of our source code, where we need to go to get the requirements file