-
Notifications
You must be signed in to change notification settings - Fork 171
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
Remove monorepo #1202
Remove monorepo #1202
Conversation
@zlav planning to resolve the conflict later, whenever I merge both my outstanding PRs. |
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.
Well that was an easy review! This cleans up a LOT
Changelog.md
Outdated
@@ -1,5 +1,8 @@ | |||
# FOSSA CLI Changelog | |||
|
|||
## v3.7.12 |
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.
Should we pair this PR with first-party license scanning and make the release 3.8? The deprecation of monorepo (breaking change) and a significant new feature seem like a good reason to bump the minor version.
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.
That works for me, as long as we're fine with waiting until the first-party license scanning is done
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.
Sounds good to me! I don't think there's anything else pressing, as long as we merge first-party license scanning before early next week then we should be all good.
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.
@spatten please just ping me (or feel free to just resolve conflicts & merge this PR) whenever that is done then!
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.
As discussed in slack, merging after all to make #1203 simpler.
Overview
Removes monorepo from FOSSA CLI, except in a couple places where we currently test for whether a project is monorepo before attempting to wait for its build or uploading a new scan.
Also removed:
fossa vps
subcommand.wiggins
embedded binary.Acceptance criteria
Monorepo no longer exists in FOSSA CLI, and meanwhile everything else still works.
I'm relying on tests and the type system to prove the latter.
Testing plan
I built the project and was able to run a simple scan, otherwise relying on tests.
Risks
No users should be depending on this functionality any longer, so this should not be risky.
References
https://fossa.atlassian.net/browse/ANE-755
Checklist
docs/
.Changelog.md
. If this PR did not mark a release, I added my changes into an# Unreleased
section at the top..fossa.yml
orfossa-deps.{json.yml}
, I updateddocs/references/files/*.schema.json
. You may also need to update these if you have added/removed new dependency type (e.g.pip
) or analysis target type (e.g.poetry
).