Skip to content

Conversation

andreafalzetti
Copy link
Contributor

Description

Update how to test/release the extension. Thanks @gtsiolis for the help!

@andreafalzetti andreafalzetti force-pushed the afalz/update-release-docs branch from 5048588 to ce5d49e Compare May 17, 2022 13:21
@andreafalzetti andreafalzetti requested a review from gtsiolis May 17, 2022 13:21
@gtsiolis gtsiolis force-pushed the afalz/update-release-docs branch from ce5d49e to 89a15ec Compare May 18, 2022 09:59
@gtsiolis
Copy link
Contributor

Looking at this now! 👀

Copy link
Contributor

@gtsiolis gtsiolis left a comment

Choose a reason for hiding this comment

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

Thanks for opening this meticulous PR, @andreafalzetti!

Changes look great and README now feels much more accurate and up-to-date.

Pushed some typo fixes and merging this. 📕


For Chrome:

1. Rename `gitpod.xpi` to `gitpod.zip`
Copy link
Contributor

Choose a reason for hiding this comment

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

praise: Thanks for using 1.s here!


### Development

To make changes and test them, the easiest way is using Gitpod itself. The `.gitpod.yml` configuration for this project installs Google Chrome and runs a VNC server to allow users to see the Desktop of the Docker container. When a workspace starts, Google Chrome is started with the Gitpod extension automatically built from source and pre-installed, therefore you can preview your changes in the ephemeral Chrome browser. 🤯
Copy link
Contributor

Choose a reason for hiding this comment

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

thought: Mind blowing indeed. Thanks @felladrin for adding this in #55.

@gtsiolis gtsiolis merged commit 9b93aea into master May 18, 2022
@andreafalzetti andreafalzetti deleted the afalz/update-release-docs branch May 18, 2022 10:21
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