Skip to content

Edit repo url after move + fix new org CI failures#1015

Merged
TuThoThai merged 5 commits into
v2.0from
v2.0_update_repo_url
May 7, 2026
Merged

Edit repo url after move + fix new org CI failures#1015
TuThoThai merged 5 commits into
v2.0from
v2.0_update_repo_url

Conversation

@TuThoThai
Copy link
Copy Markdown
Contributor

@TuThoThai TuThoThai commented May 3, 2026

After the repository was moved to its new organisation, I edited the repo URL in the following files:

  • README.md
  • CHANGELOG.md
  • xsd/netex_part_1/part1_tacticalPlanning/netex_routingConstraint_version.xsd

The change was to replace "NeTEx-CEN" with "TransmodelEcosystem"

Also: includes a fix for CI failures since moving org (#1018)

Edit the repo URL in:
- README.md
- CHANGELOG.md
- xsd/netex_part_1/part1_tacticalPlanning/netex_routingConstraint_version.xsd

The change was to replace "NeTEx-CEN" with "TransmodelEcosystem"
@TuThoThai TuThoThai requested review from skinkie and thbar May 3, 2026 14:32
@TuThoThai TuThoThai added the typo Typo (comment or mispelling) correction without any technical impact label May 3, 2026
@thbar
Copy link
Copy Markdown
Contributor

thbar commented May 6, 2026

(logically, CI is green, because nothing gets commited by the commit step here)

Comment thread .github/workflows/ci.yml
@thbar thbar changed the title Edit repo url after move Edit repo url after move + fix new org CI failures May 6, 2026
@thbar
Copy link
Copy Markdown
Contributor

thbar commented May 6, 2026

@TuThoThai to be discussed tomorrow: the fix seems to fix at least one problem. Merging it & updating the recently touched PRs, will help us verify it actually works in full.

@TuThoThai TuThoThai requested review from thbar and ue71603 May 6, 2026 20:41
@TuThoThai
Copy link
Copy Markdown
Contributor Author

@TuThoThai to be discussed tomorrow: the fix seems to fix at least one problem. Merging it & updating the recently touched PRs, will help us verify it actually works in full.

@thbar, thanks for the fix.

As soon as the PR is reviewed and approved, we can merge it and update the feature branches for the other 2 PRs.

Copy link
Copy Markdown
Contributor

@thbar thbar left a comment

Choose a reason for hiding this comment

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

LGTM!

@TuThoThai TuThoThai merged commit f55ad32 into v2.0 May 7, 2026
@TuThoThai TuThoThai deleted the v2.0_update_repo_url branch May 7, 2026 06:59
TuThoThai added a commit that referenced this pull request May 8, 2026
* Fix faulty descriptions

Documentation changes in the XSD for `FreeRecharging`, `Cage`, `RemoteControl`, `LocalControl` and `LockingMechanism`

Refer to #999 for more details

* Buggy SIRI import fix (#1001)

The SIRI import is probably coming from a SIRI copy/paste since the NeTEx occupancy is following the SIRI one.
The NaturalLanguageStringStructure is not NeTEx type, and is only used in this file, and doesn't follow NeTEx style/rules
The SIRI import is crashing some generic NeTEx (and SIRI) import in some implementation (one of them being OpRa). The reason for this crash is difficult to analyse (and depending on the validation engine), but basically related to object redefinition. As a result this import was a useless and dangerous breaking change and has to be removed.

* Edit changelog and readme (#997)

* Update changelog

* Remove redundancy with readme
* Remove duplicates in the changelog details for v1.1.0
* Correct hierarchy of changelog for v1.0.9 and v1.0.8

* Delete change_log.md

Remove redundant file

* Fix link to CHANGELOG in README

* Rename 'Change log' to 'Changelog'

* Update README.md

Co-authored-by: Thibaut Barrère <thibaut.barrere@gmail.com>

* Update readme - typo (#1010)

Update typo in the name of elements in the repo

* Edit repo url after move + fix new org CI failures (#1015)

2 parts Pull Request

Part 1: Edit the repo URL in
- README.md
- CHANGELOG.md
- xsd/netex_part_1/part1_tacticalPlanning/netex_routingConstraint_version.xsd
The change was to replace "NeTEx-CEN" with "TransmodelEcosystem"

Part 2: Fix failing CI after moving the repo
- Test CI after org migration (#1018)
- Test: break indentation to trigger CI lint commit (#1018)
- Try to fix CI (#1018): grant write permissions to workflow token
- Lint and update documentation tables

---------

Co-authored-by: Thibaut Barrère <thibaut.barrere@gmail.com>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

* Add some logic to prevent #982 from happening again (#1022)

Ensure that 
- v2.1-wip branch is up to date from v2.0 before merging any PR
- v3.0-wip branch is up to date from v2.1-wip before merging any PR

---------

Co-authored-by: Stefan de Konink <stefan@konink.de>
Co-authored-by: Christophe Duquesne <christophe.duquesne@aurigetech.com>
Co-authored-by: Thibaut Barrère <thibaut.barrere@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

typo Typo (comment or mispelling) correction without any technical impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants