Add italian-cap package - #236
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces a new Espanso hub package (italian-cap) that expands Italian CAP (postal codes) into CAP + comune via per-CAP triggers.
Changes:
- Added package README documenting installation, usage, and how duplicate CAPs are resolved to a single comune.
- Added MIT license for the package.
- Added hub manifest metadata (
_manifest.yml) for listing and discovery.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/italian-cap/0.1.0/README.md | Documents package purpose, installation, usage, and data resolution rules. |
| packages/italian-cap/0.1.0/LICENSE | Adds MIT licensing information for the package. |
| packages/italian-cap/0.1.0/_manifest.yml | Provides hub metadata (name/title/description/version/author/tags) for the package. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
smeech
requested changes
Jul 29, 2026
smeech
left a comment
smeech
left a comment
Collaborator
There was a problem hiding this comment.
A simple package of trigger/replace pairs. No script, shell or potentially malicious code. A couple of changes to make, please.
I'll run Copilot and the automated checks.
blackne0n
force-pushed
the
add-italian-cap
branch
from
July 29, 2026 13:07
ff6e6fd to
43a02bb
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (2)
packages/italian-cap/0.1.0/_manifest.yml:7
- The
homepagepoints to a branch-specific URL (.../tree/add-italian-cap) and appears to have a typo in the GitHub username (blacne0nvsblackne0n). After merging, branch links may disappear and users will be sent to a 404; consider updating this to a stable URL (e.g., default branch or a dedicated repository page for the package).
homepage: "https://github.com/blacne0n/hub/tree/add-italian-cap"
packages/italian-cap/0.1.0/README.md:22
- The README mentions that triggers are generated from an “ISTAT-derived” list, but it doesn’t include a source link or any licensing/attribution details for that dataset. Adding the specific dataset URL and its license/terms would make redistribution terms clear for downstream users.
## Data
Triggers are generated from the ISTAT-derived list of
Italian comuni with their CAP.
blackne0n
force-pushed
the
add-italian-cap
branch
from
July 29, 2026 13:19
43a02bb to
6ccccdc
Compare
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.
Add italian-cap package which converts italian CAP (CAP code) to CAP + City name.