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

adds support for theatlantic.com #376

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kemelzaidan
Copy link

Hopefully it adds support for the news site theatlantic.com. I mean hopefully because I didn't find a way to run this version of the script in my tampermonkey.

As I could understand, you process all the files in order to create a single one and load it to the browser extension. I did install the dependencies with npm i and was able to run the included makefile, but could not find a single file in the dist folder in order to try to add to my tampermonkey.

It would be awesome if you could include a CONTRIBUTING.md file in the root folder with simple instruction in order to test the changes and see if they are working. This is a project I use a lot and would be happy if I can contribute more often.

Thanks a lot!

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

1 participant