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

Jlc/pin nelc api clients #240

Merged
merged 2 commits into from
Feb 14, 2025
Merged

Jlc/pin nelc api clients #240

merged 2 commits into from
Feb 14, 2025

Conversation

johanseto
Copy link
Collaborator

Description

[feat: pin nelc api client to v0.1.0]

This is for not breaking API_clients due to the new changes in 'main' of the nelc_api_clients library.

@github-actions github-actions bot added requirements size/m m lines label labels Feb 13, 2025
@johanseto johanseto force-pushed the jlc/pin-nelc-api-clients branch from 24b3121 to 274a316 Compare February 13, 2025 17:50
@johanseto johanseto force-pushed the jlc/pin-nelc-api-clients branch from 274a316 to 0eb225d Compare February 13, 2025 19:48
@johanseto johanseto force-pushed the jlc/pin-nelc-api-clients branch from 0eb225d to 8c25371 Compare February 13, 2025 20:14
@johanseto johanseto changed the base branch from master to jlc/upgrade-test-python February 13, 2025 20:14
@johanseto johanseto requested a review from Ang-m4 February 13, 2025 20:19
@johanseto johanseto changed the base branch from jlc/upgrade-test-python to master February 14, 2025 14:10
@johanseto johanseto force-pushed the jlc/pin-nelc-api-clients branch from 8c25371 to 7efd434 Compare February 14, 2025 14:11
@johanseto johanseto requested review from DeploymentRobotEdunext and Ang-m4 and removed request for Ang-m4 February 14, 2025 14:12
Copy link
Contributor

@Ang-m4 Ang-m4 left a comment

Choose a reason for hiding this comment

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

LGTM

@johanseto johanseto merged commit 02eb091 into master Feb 14, 2025
7 checks passed
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