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

Maintain Google Colab notebook internally #2420

Closed
wants to merge 14 commits into from
Closed

Maintain Google Colab notebook internally #2420

wants to merge 14 commits into from

Conversation

ScriptTiger
Copy link
Contributor

Right now, I'm currently maintaining the notebook on my Google Drive. This update just brings the notebook into the hosts repository so it can be maintained internally. This is simply to drop the third party (myself), and this should not bring any additional burden (of significance) to the maintenance of the repository in general.

To view the intended outcome of this update, please refer to the following commit:
https://github.com/ScriptTiger/hosts/tree/1cae8718f56f45661d0103847f7c291ef0fdd5e7

As this update includes links to the StevenBlack/hosts repository which will only exist after the update is applied, using a previous commit which includes links to the ScriptTiger/hosts repository is needed since those links already exist and can be tested now.

Features:
By opening the hosts.ipynb from the GitHub repository, GitHub recognizes the format and displays it appropriately.
By clicking the "open in colab" link from either the notebook itself or README, the notebook is directly opened by Google Colab without a third-party owner/host.
Issues related specifically to the notebook can be triaged internally by the maintainers of the hosts repository, as well as keeping full transparency of the notebook internal to the repository.

@StevenBlack
Copy link
Owner

Thank you for this @ScriptTiger but I must decline this PR.

Closing.

@ScriptTiger
Copy link
Contributor Author

No worries. We can revisit it later if it's a timing issue.

@StevenBlack
Copy link
Owner

@ScriptTiger it's not a timing issue. This PR is for a personal workflow that has no bearing here.

@ScriptTiger ScriptTiger deleted the master branch August 24, 2023 18:20
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