Skip to content

Adding github action for automated submission and dependabot upkeep#51

Merged
vict0rsch merged 1 commit intovict0rsch:masterfrom
plasmo-foss:master
Feb 27, 2022
Merged

Adding github action for automated submission and dependabot upkeep#51
vict0rsch merged 1 commit intovict0rsch:masterfrom
plasmo-foss:master

Conversation

@louisgv
Copy link
Copy Markdown
Contributor

@louisgv louisgv commented Feb 24, 2022

Hi @vict0rsch!

Love PaperMemory!!! It makes note taking on arxiv to another level. I'm spreading it to my friends :D Also, I was looking for ways to contribute, and found that since your codebase doesn't have automated ci, I thought adding that might be helpful.

First, I added a dependabot config for you so it should bump dependencies version automatically. Then, I created 2 workflows in this PR:

  • automerge: automate the dependabot merging to reduce PR noise, using https://github.com/fastify/github-action-merge-dependabot
  • submit: automate the zip submission process to the Chrome/Firefox/Edge stores automatically, using bpp . It is set to run manually from the github action tab, but we can tweak it to run as frequent as you wish! The only thing you would need to create is a SUBMIT_KEYS github repository secret.

The SUBMIT_KEYS secret is a json, with the schema defined here.

Here's a sample key for you:

{
  "$schema": "https://raw.githubusercontent.com/plasmo-corp/bpp/v1/keys.schema.json",
  "chrome": {
    "clientId": "123",
    "refreshToken": "789",
    "extId": "abcd"
  },
	"firefox": {
		"apiKey": "123",
		"apiSecret": "789",
		"extId": "abcd"
	}
}

You can find instructions on how to get those keys in the schema, or if you use vscode, the schema should provide hint/intelisense when hovering over the json properties. If you need any help in setting up the keys, feel free to @ me! Otherwise if this doesn't seem necessary, feel free to close the PR :)

@vict0rsch
Copy link
Copy Markdown
Owner

That's amazing @louisgv. Thanks so much. It's a personally challenging moment so sorry for being a little laggy to respond and I'll probably take a few more days to check everything out and merge but it's great to have you contribute! We can discuss further contributions if you wish to, I've been driving this alone and mostly for my own sake so feel free to make suggestions too!

@vict0rsch vict0rsch merged commit 298eac2 into vict0rsch:master Feb 27, 2022
@louisgv
Copy link
Copy Markdown
Contributor Author

louisgv commented Feb 27, 2022

That's amazing @louisgv. Thanks so much. It's a personally challenging moment so sorry for being a little laggy to respond and I'll probably take a few more days to check everything out and merge but it's great to have you contribute! We can discuss further contributions if you wish to, I've been driving this alone and mostly for my own sake so feel free to make suggestions too!

I'm glad it was helpful :D

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