Skip to content

Fix typos#140

Merged
MetRonnie merged 1 commit intocylc:v1from
wxtim:feat.allow-install-editable
Jan 29, 2026
Merged

Fix typos#140
MetRonnie merged 1 commit intocylc:v1from
wxtim:feat.allow-install-editable

Conversation

@wxtim
Copy link
Member

@wxtim wxtim commented Jan 29, 2026

Follow-up to #136

Checklist

  • I have read the contributing instructions in README.md and have opened this against the correct branch & milestone

@wxtim wxtim requested review from MetRonnie and oliver-sanders and removed request for MetRonnie January 29, 2026 12:41
@MetRonnie

This comment was marked as resolved.

@wxtim wxtim changed the title fix - avoid adding at after fix fix - avoid adding @ if version number no present Jan 29, 2026
@wxtim wxtim changed the title fix - avoid adding @ if version number no present Avoid adding @ if version number no present Jan 29, 2026
@wxtim wxtim requested a review from MetRonnie January 29, 2026 12:58
@wxtim wxtim force-pushed the feat.allow-install-editable branch from 67ba024 to 6eb67f2 Compare January 29, 2026 13:03
@MetRonnie MetRonnie changed the title Avoid adding @ if version number no present Fix typos Jan 29, 2026
@MetRonnie MetRonnie removed the request for review from oliver-sanders January 29, 2026 13:54
@MetRonnie MetRonnie merged commit b9c6c7e into cylc:v1 Jan 29, 2026
7 checks passed
@wxtim wxtim deleted the feat.allow-install-editable branch January 29, 2026 15:29
@MetRonnie
Copy link
Member

@wxtim It seems we've still got some issues with the new format: https://github.com/cylc/release-actions/actions/runs/21570770992/job/62149466610?pr=142

pip install git+https://github.com/cylc/cylc-flow@8.4.0#egg=cylc-flow[all]
error: invalid-egg-fragment

× The 'cylc-flow[all]' egg fragment is invalid
╰─> from 'git+https://github.com/cylc/cylc-flow@8.4.0#egg=cylc-flow[all]'

hint: Try using the Direct URL requirement syntax: 'name[extra] @ URL'

@wxtim wxtim mentioned this pull request Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants