-
Notifications
You must be signed in to change notification settings - Fork 6
Bunch of small fixes for the plugin #50
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
Conversation
🦋 Changeset is good to goLatest commit: ef37f25 We got this. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
| const convertToGraph = require("../convertToGraph"); | ||
|
|
||
| describe("should convert Plugin data to graph", () => { | ||
| // I think this test is running against outdated data |
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 probably figure this one out.
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.
Yeah, we need to update the copy of the data.
* Bunch of small fixes for the plugin * Unbumping versions * Fixing tests * Adding a changeset
* Bunch of small fixes for the plugin * Unbumping versions * Fixing tests * Adding a changeset
* Bunch of small fixes for the plugin * Unbumping versions * Fixing tests * Adding a changeset
* Bunch of small fixes for the plugin * Unbumping versions * Fixing tests * Adding a changeset
Removing the requirement on
metadataand updating the docs to have even more content.