Skip to content

[CI] Remove Travis code (LRN-51189)#103

Merged
danny-beton-lrn merged 1 commit into
masterfrom
LRN-51189/feature/remove_travis
May 27, 2026
Merged

[CI] Remove Travis code (LRN-51189)#103
danny-beton-lrn merged 1 commit into
masterfrom
LRN-51189/feature/remove_travis

Conversation

@danny-beton-lrn
Copy link
Copy Markdown
Contributor

Remove Travis CI configuration and references

This PR removes Travis CI from the repository as it is no longer in use within the organization. The changes include removing the .travis.yml configuration file and the Travis CI build status badge from both the main README and the distribution package README. This is a purely cosmetic change that does not impact the SDK functionality, tests, or dependencies in any way.

@danny-beton-lrn
Copy link
Copy Markdown
Contributor Author

Auggie Code Review:

Reviewing Travis Changes
Overall this looks safe to merge: removing .travis.yml and the Travis badge lines in both README.md files won't affect the SDK runtime, Composer metadata, or tests.

One thing to flag before merging given the "remove Travis and all references" requirement: the committed release artifact learnosity_sdk-1.2.1.zip still contains the old README.md content with a Travis reference inside the archive (the repo-wide text search won't catch this because it's inside the zip). If that zip is meant to stay in-repo, it should be regenerated to reflect the updated README (or removed if it's just a stale artifact).

@danny-beton-lrn
Copy link
Copy Markdown
Contributor Author

No need to update the zip, it will be regenerated next release

@danny-beton-lrn danny-beton-lrn merged commit eb414a9 into master May 27, 2026
2 checks passed
@danny-beton-lrn danny-beton-lrn deleted the LRN-51189/feature/remove_travis branch May 27, 2026 12:53
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