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

Adds releaser #10

Merged
merged 4 commits into from
Jun 1, 2023
Merged

Adds releaser #10

merged 4 commits into from
Jun 1, 2023

Conversation

hbcarlos
Copy link
Collaborator

@hbcarlos hbcarlos commented Jun 1, 2023

No description provided.

@hbcarlos hbcarlos self-assigned this Jun 1, 2023
@hbcarlos hbcarlos merged commit 31f0d71 into jupyter-robotics:main Jun 1, 2023
@hbcarlos hbcarlos deleted the releaser branch June 1, 2023 11:16
uses: jupyterlab/maintainer-tools/.github/actions/base-setup@v1
- name: Install Dependencies
run: |
pip install -e .
Copy link
Collaborator

@jtpio jtpio Jun 1, 2023

Choose a reason for hiding this comment

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

Normally it would be better to remove this step as it is not needed anymore with releaser v2.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Having a pip install -e . only in the check release workflow but not in the "Publish Release" workflow (the one used to do the real release) can add unexpected side effects (some dev dependencies missing in the actual release workflow.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks @jtpio

hbcarlos added a commit to hbcarlos/ipylgbst that referenced this pull request Jun 1, 2023
hbcarlos added a commit that referenced this pull request Jun 1, 2023
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.

2 participants