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

Tidy up files to reflect the migration to GitHub Actions #328

Merged
merged 1 commit into from
Sep 16, 2021

Conversation

terrytangyuan
Copy link
Member

@terrytangyuan terrytangyuan commented Sep 16, 2021

@bellet
Copy link
Member

bellet commented Sep 16, 2021

Thanks! I was thinking maybe we can clean up things that are not needed anymore. Perhaps .travis.yml and build_tools/travis?

@terrytangyuan
Copy link
Member Author

Good idea. Let me do that.

Signed-off-by: Yuan Tang <terrytangyuan@gmail.com>
@bellet
Copy link
Member

bellet commented Sep 16, 2021

Not sure also what .landscape.yml is for? Looks quite old

@terrytangyuan terrytangyuan changed the title Update CI build badge to GitHub Actions Tidy up files to reflect the migration to GitHub Actions Sep 16, 2021
@terrytangyuan
Copy link
Member Author

That seems to be used for benchmarking report. @perimosocordiae Looks like https://landscape.io/ is down. Are we still using this?

Let's keep this PR specific for GitHub Actions related clean up? @bellet

Copy link
Member

@bellet bellet left a comment

Choose a reason for hiding this comment

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

Sure!

@bellet bellet merged commit f548b1e into master Sep 16, 2021
@terrytangyuan terrytangyuan deleted the terrytangyuan-patch-1 branch September 16, 2021 13:50
@perimosocordiae
Copy link
Contributor

That seems to be used for benchmarking report. @perimosocordiae Looks like https://landscape.io/ is down. Are we still using this?

Apparently not! 😄

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.

None yet

3 participants