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

Remove the on_push.js file now that publishing the website is moved to GitHub Actions #39

Merged
merged 1 commit into from
Apr 27, 2023

Conversation

timvandermeij
Copy link
Contributor

Note that this file can safely be removed because the Bot.io framework explicitly checks if this file is available and otherwise simply does nothing on push actions; see https://github.com/arturadib/botio/blob/master/lib/server.js#L387-L390.

Relates to mozilla/pdf.js#16364.

… to GitHub Actions

Note that this file can safely be removed because the Bot.io framework
explicitly checks if this file is available and otherwise simply does
nothing on push actions; see
https://github.com/arturadib/botio/blob/master/lib/server.js#L387-L390.
Copy link
Contributor

@Snuffleupagus Snuffleupagus left a comment

Choose a reason for hiding this comment

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

r=me, thank you!


@calixteman Can you please take care of pushing this to the bots?

@calixteman calixteman merged commit 1731a51 into mozilla:master Apr 27, 2023
@calixteman
Copy link
Contributor

Done. Thank you.

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.

3 participants