Implemented .well-known for all known provenant AIDs - #20
Merged
Conversation
Also added tags to .gitignore and helper scripts for the next go-around.
Removed unneeded vim swap file.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds static OOBI/.well-known discovery artifacts for known Provenant AIDs (witness controllers, schemas, and AID witness OOBIs), plus helper scripts to (re)generate/download these resources and a host-meta entry point for discovery.
Changes:
- Added
.well-known/host-meta.jsonto advertise OOBI lookup and a discovery catalog endpoint. - Added many new static OOBI resources under
.well-known/oobi/(and mirrored copies underoobi//well-known-assets/). - Added helper scripts for fetching witness OOBIs and schemas, and updated
.gitignoreto ignore.tags.
Reviewed changes
Copilot reviewed 150 out of 154 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| well-known-assets/witness/BPV5YJ03jIUdOtXkxO7dYPVzVf61IKqYa3DGCVy1NW51/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BOCWZuhoRHL_HpySDk450Shz2CNf9N5XNWmumlzvDGJj/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BO-Cc_Ylcw4HzlXPfUIasnS7BDKgrFmt4-b3kYvtIzBx/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BO_zDhNmQs-zd6yjMRiKPZjQbo78bpa9Kt9bo2e2YFUL/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BNwSX8dtJ_Q-jlSIcgaL9phC2qT-PwNy_z1p-QSFPGMg/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BNFbib-oPD6bmFCN8LnCJADWVv8QLQCATeNKbkerF-lm/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BN6qlj8h1oq0YEYPxrQ_-nY1DpFDE85KF_Go-OGq0ie1/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BMm-sMQeEGbmVJ40TV1DwPQtU1Do8MCS48FkqhDK6ggW/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BLBC0dK4vnEEMa3Gw_P9_rHow6BRmU5lIXUqxdbEKWKk/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BK5BgSYAzXvkzZU03-8Fo_eWoVdvlwQexGavi205MKQN/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BIWUdHXuBMngklyP2e_h422r67NXj8N1018Qi-EzsYNy/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BIOK5kUx3vzairte9dhqv68TtimI78TYezaFOEjrdSTb/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BIHIg-sMesHIzbLzl8r9hq4797WZ8yKBidIKUKPrmEAk/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BG6aVhGdmp-XtyvWE566MxTwAca3877ru0ni1TgBTv7c/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BFNdkltTbYsSxu3lDsKF4jWmUS54WD_Pdqxrts3_wzlF/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BDz5Zk4hRe_qm04TE_Z2lT5dyMyjqCH-BTvPOzyZUBSH/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/witness/BDveFelREBqEeBaP4jS3gJow57OpPyLUKcC_xmQn4OKg/index.json | Adds witness controller OOBI payload (static asset). |
| well-known-assets/schema/EPhWFgeOy8g7yRy-Xtyvbdieqvl_3YVXNHMgTEZuJOWh/index.json | Publishes AI-coder-license JSON schema (static asset). |
| well-known-assets/schema/ENPXp1vQzRF6JwIuS-mp2U8Uf1MoADoP_GqQ62VsDZWY/index.json | Publishes Legal Entity vLEI JSON schema (static asset). |
| well-known-assets/schema/EMhvwOlyEJ9kN4PrwCpr9Jsv7TxPhiYveZ0oP3lJzdEi/index.json | Publishes iXBRL attestation JSON schema (static asset). |
| well-known-assets/schema/EKR1nFV0JvRhc4xkhzlZJfpN_taaspipcZWWSLj37Fdk/index.json | Publishes Aegis vetting JSON schema (static asset). |
| well-known-assets/schema/EJxFPpyDRV-W6O2Vtjdy2K90ltWmQK8l1jePw5YOo_Ft/index.json | Publishes Data Attestation JSON schema (static asset). |
| well-known-assets/schema/EG68irpfVX667KCLwG85Cn1Mp3sCe38ftARyQJrxP2kF/index.json | Publishes Proof-of-Control JSON schema (static asset). |
| well-known-assets/schema/EFvHYHX0cUx9sdjxZOr9fpPcQKdzRNFH42D8R29p7lAH/index.json | Publishes Bind Key JSON schema (static asset). |
| well-known-assets/schema/ECYorXkheU7YsXZkYLGtvBOxEZ6alS5H5FJRn0tgDXV0/index.json | Publishes iXBRL D6 attestation JSON schema (static asset). |
| well-known-assets/schema/EBNaNu-M9P5cgrnfl2Fvymy4E_jvxxyjb70PRtiANlJy/index.json | Publishes OOR role vLEI JSON schema (static asset). |
| well-known-assets/schema/EBfdlu8R27Fbx-ehrqwImnK-8Cm79sqbAQ4MmvEAYqao/index.json | Publishes Qualified vLEI Issuer JSON schema (static asset). |
| well-known-assets/schema/EBCnd7qk82wLBOgFukdmsdkksAuPpmzt5-eg9YKWWP3j/index.json | Publishes AI user COCA JSON schema (static asset). |
| well-known-assets/aid/EDP1vHcw_wc4M__Fj53-cJaBnZZASd-aMTaSyWEQ-PC2/index.json | Publishes AID witness OOBI (static asset). |
| well-known-assets/aid/ED88Jn6CnWpNbSYz6vp9DOSpJH2_Di5MSwWTf1l34JJm/index.json | Publishes AID witness OOBI (static asset). |
| scripts/witness-urls-from-witness-version-tracker | Adds source list of witness controller URLs used by helper scripts. |
| scripts/schema-getter.sh | Adds script for populating schema static directory from registry. |
| scripts/get_witness_oobis.sh | Adds script to download witness controller OOBIs into directories. |
| oobi/EPhWFgeOy8g7yRy-Xtyvbdieqvl_3YVXNHMgTEZuJOWh/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/ENPXp1vQzRF6JwIuS-mp2U8Uf1MoADoP_GqQ62VsDZWY/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EMhvwOlyEJ9kN4PrwCpr9Jsv7TxPhiYveZ0oP3lJzdEi/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EKR1nFV0JvRhc4xkhzlZJfpN_taaspipcZWWSLj37Fdk/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EKA57bKBKxr_kN7iN5i7lMUxpMG-s19dRcmov1iDxz-E/index.json | Adds OOR authorization schema OOBI payload under oobi/. |
| oobi/EJxFPpyDRV-W6O2Vtjdy2K90ltWmQK8l1jePw5YOo_Ft/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EG68irpfVX667KCLwG85Cn1Mp3sCe38ftARyQJrxP2kF/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EFvHYHX0cUx9sdjxZOr9fpPcQKdzRNFH42D8R29p7lAH/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EDP1vHcw_wc4M__Fj53-cJaBnZZASd-aMTaSyWEQ-PC2/index.json | Mirrors AID witness OOBI payload under oobi/. |
| oobi/ED88Jn6CnWpNbSYz6vp9DOSpJH2_Di5MSwWTf1l34JJm/index.json | Mirrors AID witness OOBI payload under oobi/. |
| oobi/ECYorXkheU7YsXZkYLGtvBOxEZ6alS5H5FJRn0tgDXV0/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EBNaNu-M9P5cgrnfl2Fvymy4E_jvxxyjb70PRtiANlJy/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EBfdlu8R27Fbx-ehrqwImnK-8Cm79sqbAQ4MmvEAYqao/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/EBCnd7qk82wLBOgFukdmsdkksAuPpmzt5-eg9YKWWP3j/index.json | Mirrors schema OOBI payload under oobi/. |
| oobi/BPV5YJ03jIUdOtXkxO7dYPVzVf61IKqYa3DGCVy1NW51/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BOCWZuhoRHL_HpySDk450Shz2CNf9N5XNWmumlzvDGJj/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BO-Cc_Ylcw4HzlXPfUIasnS7BDKgrFmt4-b3kYvtIzBx/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BO_zDhNmQs-zd6yjMRiKPZjQbo78bpa9Kt9bo2e2YFUL/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BNwSX8dtJ_Q-jlSIcgaL9phC2qT-PwNy_z1p-QSFPGMg/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BNFbib-oPD6bmFCN8LnCJADWVv8QLQCATeNKbkerF-lm/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BN6qlj8h1oq0YEYPxrQ_-nY1DpFDE85KF_Go-OGq0ie1/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BMm-sMQeEGbmVJ40TV1DwPQtU1Do8MCS48FkqhDK6ggW/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BLBC0dK4vnEEMa3Gw_P9_rHow6BRmU5lIXUqxdbEKWKk/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BK5BgSYAzXvkzZU03-8Fo_eWoVdvlwQexGavi205MKQN/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BIWUdHXuBMngklyP2e_h422r67NXj8N1018Qi-EzsYNy/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BIOK5kUx3vzairte9dhqv68TtimI78TYezaFOEjrdSTb/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BIHIg-sMesHIzbLzl8r9hq4797WZ8yKBidIKUKPrmEAk/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BG6aVhGdmp-XtyvWE566MxTwAca3877ru0ni1TgBTv7c/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BFNdkltTbYsSxu3lDsKF4jWmUS54WD_Pdqxrts3_wzlF/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BDz5Zk4hRe_qm04TE_Z2lT5dyMyjqCH-BTvPOzyZUBSH/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BDveFelREBqEeBaP4jS3gJow57OpPyLUKcC_xmQn4OKg/index.json | Mirrors witness controller OOBI payload under oobi/. |
| oobi/BCf29L_7oQtU8WUXEV2Bi5sf7WoxnGyX7sgJSym-p4Pp/index.json | Adds dev witness controller OOBI payload under oobi/. |
| oobi/BBcNDQkX6FTFstVx3DDFCEcPN9_-gSkIDDrtt3uIHRiV/index.json | Adds asia witness controller OOBI payload under oobi/. |
| .well-known/oobi/EPhWFgeOy8g7yRy-Xtyvbdieqvl_3YVXNHMgTEZuJOWh/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/ENPXp1vQzRF6JwIuS-mp2U8Uf1MoADoP_GqQ62VsDZWY/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EMhvwOlyEJ9kN4PrwCpr9Jsv7TxPhiYveZ0oP3lJzdEi/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EKR1nFV0JvRhc4xkhzlZJfpN_taaspipcZWWSLj37Fdk/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EJxFPpyDRV-W6O2Vtjdy2K90ltWmQK8l1jePw5YOo_Ft/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EG68irpfVX667KCLwG85Cn1Mp3sCe38ftARyQJrxP2kF/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EFvHYHX0cUx9sdjxZOr9fpPcQKdzRNFH42D8R29p7lAH/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EDP1vHcw_wc4M__Fj53-cJaBnZZASd-aMTaSyWEQ-PC2/index.json | Publishes AID witness OOBI under the public .well-known path. |
| .well-known/oobi/ED88Jn6CnWpNbSYz6vp9DOSpJH2_Di5MSwWTf1l34JJm/index.json | Publishes AID witness OOBI under the public .well-known path. |
| .well-known/oobi/ECYorXkheU7YsXZkYLGtvBOxEZ6alS5H5FJRn0tgDXV0/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EBNaNu-M9P5cgrnfl2Fvymy4E_jvxxyjb70PRtiANlJy/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EBfdlu8R27Fbx-ehrqwImnK-8Cm79sqbAQ4MmvEAYqao/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/EBCnd7qk82wLBOgFukdmsdkksAuPpmzt5-eg9YKWWP3j/index.json | Publishes schema OOBI under the public .well-known path. |
| .well-known/oobi/BPV5YJ03jIUdOtXkxO7dYPVzVf61IKqYa3DGCVy1NW51/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BOCWZuhoRHL_HpySDk450Shz2CNf9N5XNWmumlzvDGJj/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BO-Cc_Ylcw4HzlXPfUIasnS7BDKgrFmt4-b3kYvtIzBx/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BO_zDhNmQs-zd6yjMRiKPZjQbo78bpa9Kt9bo2e2YFUL/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BNwSX8dtJ_Q-jlSIcgaL9phC2qT-PwNy_z1p-QSFPGMg/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BNFbib-oPD6bmFCN8LnCJADWVv8QLQCATeNKbkerF-lm/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BN6qlj8h1oq0YEYPxrQ_-nY1DpFDE85KF_Go-OGq0ie1/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BMm-sMQeEGbmVJ40TV1DwPQtU1Do8MCS48FkqhDK6ggW/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BLBC0dK4vnEEMa3Gw_P9_rHow6BRmU5lIXUqxdbEKWKk/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BK5BgSYAzXvkzZU03-8Fo_eWoVdvlwQexGavi205MKQN/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BIWUdHXuBMngklyP2e_h422r67NXj8N1018Qi-EzsYNy/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BIOK5kUx3vzairte9dhqv68TtimI78TYezaFOEjrdSTb/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BIHIg-sMesHIzbLzl8r9hq4797WZ8yKBidIKUKPrmEAk/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BG6aVhGdmp-XtyvWE566MxTwAca3877ru0ni1TgBTv7c/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BFNdkltTbYsSxu3lDsKF4jWmUS54WD_Pdqxrts3_wzlF/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BDz5Zk4hRe_qm04TE_Z2lT5dyMyjqCH-BTvPOzyZUBSH/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BDveFelREBqEeBaP4jS3gJow57OpPyLUKcC_xmQn4OKg/index.json | Publishes witness controller OOBI under the public .well-known path. |
| .well-known/oobi/BCf29L_7oQtU8WUXEV2Bi5sf7WoxnGyX7sgJSym-p4Pp/index.json | Publishes dev witness controller OOBI under the public .well-known path. |
| .well-known/host-meta.json | Adds host-meta discovery document for locating OOBI resources and the catalog. |
| .gitignore | Ignores .tags file(s). |
Suppressed comments (2)
scripts/witness-urls-from-witness-version-tracker:10
- These URLs have trailing whitespace at the end of the line. When consumed by scripts (e.g.,
get_witness_oobis.sh), the extra space becomes part of the URL and can causecurlto fail with an invalid URL / 404.
Trim trailing whitespace (and ideally ensure the file is whitespace-clean via an editorconfig/pre-commit step).
scripts/witness-urls-from-witness-version-tracker:15
- These URLs have trailing whitespace at the end of the line. When consumed by scripts (e.g.,
get_witness_oobis.sh), the extra space becomes part of the URL and can causecurlto fail with an invalid URL / 404.
Trim trailing whitespace (and ideally ensure the file is whitespace-clean via an editorconfig/pre-commit step).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 150 out of 154 changed files in this pull request and generated no new comments.
Suppressed comments (5)
well-known-assets/schema/EG68irpfVX667KCLwG85Cn1Mp3sCe38ftARyQJrxP2kF/index.json:1
- The attributes schema requires a "minutes" field, but the properties block does not define it. Also, "exclusiveMin" is not a valid JSON Schema keyword in draft 2020-12 (use "exclusiveMinimum" or "minimum"), so validators will ignore this constraint.
scripts/get_witness_oobis.sh:5 - This script lacks a shebang/strict mode and uses relative paths to the URL list, so it will break when run from outside the scripts directory. It also uses plain
mkdir(fails on reruns) andcurlwithout-f(errors can be silently written to index.json).
scripts/schema-getter.sh:12 - This script writes downloaded schemas into directories relative to the current working directory (e.g., creating .//index.json). When run from the repo root (which is a common default), it will scatter generated directories across the project instead of updating a predictable location under well-known assets.
well-known-assets/schema/EMhvwOlyEJ9kN4PrwCpr9Jsv7TxPhiYveZ0oP3lJzdEi/index.json:1 - Typo in schema description: "contai0ning" (with a zero) should be "containing". This appears in the ECR edge description and is likely a copy/paste artifact.
well-known-assets/schema/ECYorXkheU7YsXZkYLGtvBOxEZ6alS5H5FJRn0tgDXV0/index.json:1 - Typo in schema description: "contai0ning" (with a zero) should be "containing". This appears in the ECR edge description and is likely a copy/paste artifact.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 150 out of 154 changed files in this pull request and generated no new comments.
Suppressed comments (3)
scripts/witness-urls-from-witness-version-tracker:4
- This URL line has trailing whitespace after
/controller. The companionget_witness_oobis.shderives directory names via asedpattern that expects the line to end at/controller, so the extra space prevents the match and can yield an incorrectoobi_dir(e.g., the full URL), breakingmkdir -p/ downloads. Please remove the trailing space here (and similarly on other affected lines).
scripts/get_witness_oobis.sh:7 - The
sedextraction assumes each URL line ends exactly with/controller. If any line has trailing whitespace (currently true inwitness-urls-from-witness-version-tracker), the substitution won't match andoobi_dircan become the full URL, causingmkdir/download failures. Make the pattern tolerate trailing whitespace and avoid greedy.*so the directory is reliably the AID segment.
well-known-assets/schema/EG68irpfVX667KCLwG85Cn1Mp3sCe38ftARyQJrxP2kF/index.json:1 - This schema declares JSON Schema draft 2020-12, but uses the keyword
exclusiveMin, which is not a valid draft 2020-12 keyword (the correct keyword isexclusiveMinimum). As written, validators will ignore the constraint, sobecan incorrectly accept 0 or negative values.
…nant-dev/provenant.net into feature/implement-gleif-well-known
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 151 out of 154 changed files in this pull request and generated no new comments.
Suppressed comments (6)
WELL-KNOWN.md:21
- This paragraph has a subject/verb agreement issue ("These ... is") and a couple of proper-noun/ellipsis grammar issues that make the instructions harder to read.
scripts/witness-urls-from-witness-version-tracker:4 - Trailing whitespace at end of line can create noisy diffs and sometimes causes subtle issues when other tooling consumes the file. Please remove the extra space after "/controller".
scripts/witness-urls-from-witness-version-tracker:10 - Trailing whitespace at end of line can create noisy diffs and sometimes causes subtle issues when other tooling consumes the file. Please remove the extra space after "/controller".
scripts/witness-urls-from-witness-version-tracker:15 - Trailing whitespace at end of line can create noisy diffs and sometimes causes subtle issues when other tooling consumes the file. Please remove the extra space after "/controller".
WELL-KNOWN.md:6 - Grammar: "Its" should be "It's", and "a fixed directory structures" is ungrammatical (singular/plural mismatch).
This issue also appears on line 19 of the same file.
scripts/get_witness_oobis.sh:7
- This script assumes it is run from the
scripts/directory (it referenceswitness-urls-from-witness-version-trackerwithout a path). Running it from the repo root (e.g../scripts/get_witness_oobis.sh) will fail because the input file won't be found. Resolve the input path relative to the script location so it works regardless of current working directory.
Contributor
Author
|
Gonna merge this since no comments and things look good. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Also added tags to .gitignore and helper scripts for the next go-around.