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
[student-libraries] Allow exporting nested libraries #31971
Conversation
@@ -100,11 +100,17 @@ class LibraryCreationDialog extends React.Component { | |||
this.setState({publishState: PublishState.NO_FUNCTIONS}); | |||
return; | |||
} | |||
let librarySource = response.source; | |||
response.libraries && |
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.
nit: I think this would be clearer as if (response.libraries) {...}
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.
👍
let libraryName = 'name'; | ||
let source = 'function foo() {}'; | ||
beforeEach(() => { | ||
getJSLintAnnotationsStub = sinon.stub( |
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.
nit: I'm not sure if there's a better name, but getJSLintAnnotationsStub
to me reads like it's a function to get a stub for JSLintAnnotations
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 tried finding a better name, but couldn't come up with anything (stubGetJSLintAnnotations has the same problem). I'm going to leave it as-is.
Description
Previously, students who exported a library that used a different library would cause breaks in projects that imported their library. This fixes that issue. Students can now import a library into a project and publish that project as a library.
Links
Testing story
Updated tests
Reviewer Checklist: