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

Run prettier on everything #563

Merged
merged 1 commit into from
Jun 21, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: Release Build
on:
push:
branches:
- 'release/**'
- "release/**"

jobs:
build-for-store:
Expand All @@ -13,7 +13,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
with:
node-version: '14.x'
node-version: "14.x"
- uses: bahmutov/npm-install@v1
- run: npm run build
env:
Expand All @@ -36,7 +36,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
with:
node-version: '14.x'
node-version: "14.x"
- uses: bahmutov/npm-install@v1
- run: npm run build
env:
Expand All @@ -57,4 +57,4 @@ jobs:
env:
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
SLACK_CHANNEL: "release"
SLACK_MESSAGE: 'Chrome release build succeeded :rocket:'
SLACK_MESSAGE: "Chrome release build succeeded :rocket:"
58 changes: 29 additions & 29 deletions .github/workflows/codeql-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,12 @@ name: "CodeQL"

on:
push:
branches: [ main ]
branches: [main]
pull_request:
# The branches below must be a subset of the branches above
branches: [ main ]
branches: [main]
schedule:
- cron: '22 3 * * 0'
- cron: "22 3 * * 0"

jobs:
analyze:
Expand All @@ -28,40 +28,40 @@ jobs:
strategy:
fail-fast: false
matrix:
language: [ 'javascript' ]
language: ["javascript"]
# CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python' ]
# Learn more:
# https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#changing-the-languages-that-are-analyzed

steps:
- name: Checkout repository
uses: actions/checkout@v2
- name: Checkout repository
uses: actions/checkout@v2

# Initializes the CodeQL tools for scanning.
- name: Initialize CodeQL
uses: github/codeql-action/init@v1
with:
languages: ${{ matrix.language }}
# If you wish to specify custom queries, you can do so here or in a config file.
# By default, queries listed here will override any specified in a config file.
# Prefix the list here with "+" to use these queries and those in the config file.
# queries: ./path/to/local/query, your-org/your-repo/queries@main
# Initializes the CodeQL tools for scanning.
- name: Initialize CodeQL
uses: github/codeql-action/init@v1
with:
languages: ${{ matrix.language }}
# If you wish to specify custom queries, you can do so here or in a config file.
# By default, queries listed here will override any specified in a config file.
# Prefix the list here with "+" to use these queries and those in the config file.
# queries: ./path/to/local/query, your-org/your-repo/queries@main

# Autobuild attempts to build any compiled languages (C/C++, C#, or Java).
# If this step fails, then you should remove it and run the build manually (see below)
- name: Autobuild
uses: github/codeql-action/autobuild@v1
# Autobuild attempts to build any compiled languages (C/C++, C#, or Java).
# If this step fails, then you should remove it and run the build manually (see below)
- name: Autobuild
uses: github/codeql-action/autobuild@v1

# ℹ️ Command-line programs to run using the OS shell.
# 📚 https://git.io/JvXDl
# ℹ️ Command-line programs to run using the OS shell.
# 📚 https://git.io/JvXDl

# ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
# and modify them (or add more) to build your code if your project
# uses a compiled language
# ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
# and modify them (or add more) to build your code if your project
# uses a compiled language

#- run: |
# make bootstrap
# make release
#- run: |
# make bootstrap
# make release

- name: Perform CodeQL Analysis
uses: github/codeql-action/analyze@v1
- name: Perform CodeQL Analysis
uses: github/codeql-action/analyze@v1
6 changes: 3 additions & 3 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
with:
node-version: '14.x'
node-version: "14.x"
- uses: bahmutov/npm-install@v1
- run: npm test

Expand All @@ -21,7 +21,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
with:
node-version: '14.x'
node-version: "14.x"
- uses: bahmutov/npm-install@v1
- run: npm run build
env:
Expand Down Expand Up @@ -54,7 +54,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
with:
node-version: '14.x'
node-version: "14.x"
- uses: bahmutov/npm-install@v1
- run: npm run build:scripts
- run: node scripts/bin/headers.js
Expand Down
2 changes: 0 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,13 @@ repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v3.2.0
hooks:
- id: check-yaml
- id: end-of-file-fixer
- id: trailing-whitespace
- id: detect-private-key
- repo: https://github.com/pre-commit/mirrors-prettier
rev: v2.2.1
hooks:
- id: prettier
files: \.(js|tsx|ts|jsx|scss)$
# - repo: https://github.com/pre-commit/mirrors-eslint
# rev: v7.10.0
# hooks:
Expand Down
24 changes: 11 additions & 13 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,26 +8,24 @@ you agree to the corporate contributor license agreement.

## Did you find a bug?

*If you are reporting a security bug, please contact the maintainers privately
by emailing todd@pixiebrix.com.*
_If you are reporting a security bug, please contact the maintainers privately
by emailing todd@pixiebrix.com._

Check if the bug has already been reported by searching on GitHub under [Issues](https://github.com/pixiebrix/pixiebrix-extension/issues).

* If an issue already exists, give the first comment a "thumbs up" to indicate you're also having the problem.
When possible, add a comment with any additional context that's not already covered in the discussion.

* If you're not able to find an existing issue, [open a new issue](https://github.com/pixiebrix/pixiebrix-extension/issues/new).
Provide information about the what you were trying to do, the expected behavior, and the actual behavior.
- If an issue already exists, give the first comment a "thumbs up" to indicate you're also having the problem.
When possible, add a comment with any additional context that's not already covered in the discussion.

- If you're not able to find an existing issue, [open a new issue](https://github.com/pixiebrix/pixiebrix-extension/issues/new).
Provide information about the what you were trying to do, the expected behavior, and the actual behavior.

## Feature Pull Requests

* If it's a major new feature, first create a new issue detailing what you're trying
to accomplish and why.
* Once you get the go ahead from the project maintainers: fork the repository,
create a branch with the issue # in the name, run the [pre-commit](https://pre-commit.com/)
hooks, and submit the PR.

- If it's a major new feature, first create a new issue detailing what you're trying
to accomplish and why.
- Once you get the go ahead from the project maintainers: fork the repository,
create a branch with the issue # in the name, run the [pre-commit](https://pre-commit.com/)
hooks, and submit the PR.

## Adding Bricks, Foundations, and Services

Expand Down
12 changes: 6 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,18 +8,18 @@ registry of bricks and pre-made blueprints. You can create a PixieBrix account t
support for team features, such as shared service configurations and team bricks.

| [<img src="https://raw.githubusercontent.com/alrra/browser-logos/master/src/firefox/firefox_48x48.png" alt="Firefox" width="24px" height="24px" />](http://godban.github.io/browsers-support-badges/)<br/>Firefox | [<img src="https://raw.githubusercontent.com/alrra/browser-logos/master/src/chrome/chrome_48x48.png" alt="Chrome" width="24px" height="24px" />](http://godban.github.io/browsers-support-badges/)<br/>Chrome |
| --------- | --------- |
| last 2 versions| last 2 versions
| ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| last 2 versions | last 2 versions |

## Example Uses

Whenever you notice yourself switching back and forth between browser tabs, it's
a great time to use PixieBrix:

* Add a button to Apartments.com to send your favorite apartment listings to a Google sheet
* Show Google Scholar results on a person's LinkedIn profile
* Embed a [YouTube video](https://www.youtube.com/watch?v=dQw4w9WgXcQ) in your
employee training site
- Add a button to Apartments.com to send your favorite apartment listings to a Google sheet
- Show Google Scholar results on a person's LinkedIn profile
- Embed a [YouTube video](https://www.youtube.com/watch?v=dQw4w9WgXcQ) in your
employee training site

<img src="https://raw.githubusercontent.com/pixiebrix/pixiebrix-extension/main/img/demo.gif" alt="PixieBrix demo GIF">

Expand Down
10 changes: 3 additions & 7 deletions browsers/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,8 @@
"contextMenus"
],
"devtools_page": "devtools.html",
"externally_connectable": {
"matches": [
"https://app.pixiebrix.com/*"
]
"externally_connectable": {
"matches": ["https://app.pixiebrix.com/*"]
},
"web_accessible_resources": [
"css/*",
Expand All @@ -55,9 +53,7 @@
"default_title": "PixieBrix"
},
"background": {
"scripts": [
"background.js"
]
"scripts": ["background.js"]
},
"options_ui": {
"page": "options.html",
Expand Down
25 changes: 11 additions & 14 deletions browsers/src/action.html
Original file line number Diff line number Diff line change
Expand Up @@ -15,23 +15,20 @@
~ along with this program. If not, see <https://www.gnu.org/licenses/>.
-->

<!DOCTYPE html PUBLIC"-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<!DOCTYPE html PUBLIC"-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta content="text/html;charset=utf-8" http-equiv="Content-Type">
<meta content="utf-8" http-equiv="encoding">
<head>
<meta content="text/html;charset=utf-8" http-equiv="Content-Type" />
<meta content="utf-8" http-equiv="encoding" />
<title>PixieBrix: Page Panel</title>
<link rel="stylesheet" href="action.css" />
</head>
<body>
<div id="container">
<div class="action-sidebar-loader">
<div>
Loading PixieBrix&nbsp;&nbsp;🚀
</div>
</head>
<body>
<div id="container">
<div class="action-sidebar-loader">
<div>Loading PixieBrix&nbsp;&nbsp;🚀</div>
</div>
</div>
</div>
<script src="action.js"></script>
</body>
</body>
</html>
9 changes: 4 additions & 5 deletions browsers/src/devtools.html
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,9 @@
~ You should have received a copy of the GNU General Public License
~ along with this program. If not, see <https://www.gnu.org/licenses/>.
-->
<!DOCTYPE html PUBLIC"-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<!DOCTYPE html PUBLIC"-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<body>
<script src="devtools.js"></script>
</body>
<body>
<script src="devtools.js"></script>
</body>
</html>
61 changes: 32 additions & 29 deletions browsers/src/devtoolsPanel.html
Original file line number Diff line number Diff line change
Expand Up @@ -14,41 +14,44 @@
~ You should have received a copy of the GNU General Public License
~ along with this program. If not, see <https://www.gnu.org/licenses/>.
-->
<!doctype html>
<!DOCTYPE html>
<html style="display: flex">
<head>
<meta charset="utf8">
<head>
<meta charset="utf8" />
<style>
html {
display: flex;
height: 100%;
}
body {
margin: 0;
padding: 0;
height: 100%;
flex: 1;
display: flex;
}
#container {
display: flex;
flex: 1;
width: 100%;
height: 100vh;
position: fixed;
top: 0;
left: 0;
right: 0;
bottom: 0;
}
html {
display: flex;
height: 100%;
}
body {
margin: 0;
padding: 0;
height: 100%;
flex: 1;
display: flex;
}
#container {
display: flex;
flex: 1;
width: 100%;
height: 100vh;
position: fixed;
top: 0;
left: 0;
right: 0;
bottom: 0;
}
</style>

<link rel="stylesheet" href="devtoolsPanel.css" />
</head>
<body>
</head>
<body>
<!-- main react mount point -->
<div id="container">🚀 Loading PixieBrix devtools.</div>
<script src="devtoolsPanel.js"></script>
<script async src="https://apis.google.com/js/client.js?onload=onGAPILoad"></script>
</body>
<script
async
src="https://apis.google.com/js/client.js?onload=onGAPILoad"
></script>
</body>
</html>
16 changes: 9 additions & 7 deletions browsers/src/frame.css
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,14 @@
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*/

body { margin: 0; }
body {
margin: 0;
}
iframe {
position: absolute;
top: 0;
left: 0;
width: 100%;
height: 100%;
border: 0;
position: absolute;
top: 0;
left: 0;
width: 100%;
height: 100%;
border: 0;
}
Loading