Skip to content
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

Fix releaser job #82

Merged
merged 5 commits into from
Feb 8, 2023
Merged

Conversation

fcollonval
Copy link
Member

@fcollonval fcollonval commented Feb 8, 2023

This fixes the jupyter releaser job. Actually the culprit was the installation package step.

Configuration to include the packages files in sdist without the transient build files is a bit tricky. The final solution is to call clean:lib after build:prod in the build python step.

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@de425bc). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@          Coverage Diff           @@
##             main     #82   +/-   ##
======================================
  Coverage        ?   0.00%           
======================================
  Files           ?       3           
  Lines           ?     290           
  Branches        ?       0           
======================================
  Hits            ?       0           
  Misses          ?     290           
  Partials        ?       0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@fcollonval fcollonval changed the title Try fixing releaser job Fix releaser job Feb 8, 2023
@fcollonval fcollonval marked this pull request as ready for review February 8, 2023 15:59
Copy link
Member

@hbcarlos hbcarlos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @fcollonval!!

@hbcarlos hbcarlos merged commit 7bf96bb into jupyterlab:main Feb 8, 2023
@fcollonval fcollonval deleted the fix/jupyter-releaser branch February 8, 2023 16:52
@fcollonval
Copy link
Member Author

cc @blink1073 FYI the failure of the jupyter-releaser was due to the step installing the package in the job before executing the check-release action.

@blink1073
Copy link
Contributor

Ah, understood, thanks for following up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants