Skip to content
This repository was archived by the owner on Sep 13, 2023. It is now read-only.

Add pylint #36

Merged
merged 24 commits into from
Oct 6, 2021
Merged

Add pylint #36

merged 24 commits into from
Oct 6, 2021

Conversation

aguschin
Copy link
Contributor

No description provided.

@aguschin aguschin self-assigned this Sep 30, 2021
@aguschin aguschin marked this pull request as draft September 30, 2021 17:33
@aguschin aguschin mentioned this pull request Oct 1, 2021
Closed
@aguschin aguschin marked this pull request as ready for review October 1, 2021 13:15
@aguschin aguschin requested a review from a team October 1, 2021 13:15
@aguschin aguschin marked this pull request as draft October 1, 2021 13:15
@mike0sv mike0sv self-assigned this Oct 1, 2021
@mike0sv mike0sv added this to the Alpha Release milestone Oct 1, 2021
mike0sv and others added 7 commits October 1, 2021 18:14
@mike0sv
Pylint ignores and todos
030b865 
Signed-off-by: mike0sv <mike0sv@gmail.com>
@aguschin @mike0sv
Update LICENSE
ef6c32d
@aguschin @mike0sv
Create CODEOWNERS
79999c1
@mike0sv @aguschin
Specify issues for todos (#46)
f3e3614 
* Specify issues for todos

Signed-off-by: mike0sv <mike0sv@gmail.com>

* create few issues and add links to them in code, making tests work with `example-mlem` repo

Co-authored-by: Alexander Guschin <1aguschin@gmail.com>
@aguschin @mike0sv
Add action to release on PyPi (#50)
fcaefa4 
* add GHA to release on pypi, fix some tests
@aguschin @mike0sv
add comment-bot workflow (#52)
c92b534
@mike0sv
Pylint fixes WIP
866b5e3 
Signed-off-by: mike0sv <mike0sv@gmail.com>
@aguschin aguschin added the 🏡 housekeeping Fighting technical debt and improving dev process label Oct 4, 2021
@aguschin aguschin marked this pull request as ready for review October 4, 2021 12:45
@mike0sv mike0sv changed the base branch from main to extend-testing October 4, 2021 12:49
@mike0sv mike0sv changed the base branch from extend-testing to main October 4, 2021 12:49
mike0sv and others added 5 commits October 4, 2021 15:49
@mike0sv
Merge branch 'main' into add-pylint
7f8bdc3 
# Conflicts:
#	mlem/api/commands.py
#	mlem/contrib/xgboost.py
#	mlem/core/dataset_type.py
#	mlem/core/requirements.py
#	mlem/utils/root.py
@aguschin
Merge branch 'main' into add-pylint
b1bd464
@aguschin
writing some docstrings and help messages in cli
802f42a
@aguschin
fix some Pylance warnings
83757cb
@aguschin
fixing issues raised by pylint
31da9f2
@codecov
Copy link

codecov bot commented Oct 5, 2021 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (main@de80df0). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 60c88e4 differs from pull request most recent head 0244228. Consider uploading reports for the commit 0244228 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage        ?   75.43%           
=======================================
  Files           ?       52           
  Lines           ?     2960           
  Branches        ?        0           
=======================================
  Hits            ?     2233           
  Misses          ?      727           
  Partials        ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de80df0...0244228. Read the comment docs.

@aguschin aguschin merged commit 1442e3e into main Oct 6, 2021
@aguschin aguschin deleted the add-pylint branch October 6, 2021 17:00
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers
No reviews
Assignees

@mike0sv mike0sv

@aguschin aguschin

Labels
🏡 housekeeping Fighting technical debt and improving dev process
Projects
None yet
Milestone
Alpha Release
Development

Successfully merging this pull request may close these issues.
2 participants
@aguschin @mike0sv