Skip to content

Commit eedc0ea

Browse files
authored
Enable checks for links, linking and spelling (#3)
1 parent c54c5f9 commit eedc0ea

File tree

5 files changed

+143
-77
lines changed

5 files changed

+143
-77
lines changed
+50
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
on:
2+
workflow_dispatch:
3+
pull_request:
4+
branches:
5+
- main
6+
7+
name: "Check spelling, linting and links"
8+
9+
jobs:
10+
check-markdown:
11+
runs-on: ubuntu-latest
12+
name: "Check spelling, linting and links"
13+
steps:
14+
- uses: actions/checkout@master
15+
- run: |
16+
wget https://github.com/qir-alliance/.github/archive/refs/heads/main.zip -O clone.zip &&
17+
unzip clone.zip && mv .github-main out && rm clone.zip &&
18+
mv out/workflow-templates/utils/ utils && rm -r out
19+
shell: bash
20+
- run: |
21+
Get-ChildItem (Join-Path "utils" "problem-matchers") `
22+
| ForEach-Object {
23+
Write-Host "Adding matcher $_...";
24+
Write-Host "::add-matcher::$_";
25+
}
26+
shell: pwsh
27+
- name: markdownlint-cli
28+
# cspell:words nosborn
29+
uses: nosborn/github-action-markdown-cli@v2.0.0
30+
with:
31+
files: .
32+
config_file: 'utils/md_lint_config.yml'
33+
- name: Markdown link check
34+
# cspell:words gaurav
35+
uses: gaurav-nelson/github-action-markdown-link-check@v1
36+
with:
37+
use-verbose-mode: 'yes'
38+
if: ${{ success() || failure() }}
39+
- name: "Checkout"
40+
uses: actions/checkout@v2
41+
- name: "Check spelling in changed files"
42+
# cspell:words streetsidesoftware
43+
uses: streetsidesoftware/cspell-action@v1.2.4
44+
with:
45+
inline: warning
46+
# Only warn on spelling errors, since there's likely a lot of false
47+
# positives with terminology in quantum computing.
48+
strict: false
49+
incremental_files_only: true
50+
config: 'utils/spelling_allowlist.json'

logo/README.md

+5-1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
11
# Usage Rights
22

3-
The logo(s) and icon(s) in this folder are owned by the QIR Alliance and are available for your download and use royalty-free. By downloading an image, the QIR Alliance grants you, and you accept, a non-exclusive license to use it. All right, title and interest in the image, including the copyright therein, is retained by the QIR Alliance.
3+
The logo(s) and icon(s) in this folder are owned by the QIR Alliance and are
4+
available for your download and use royalty-free. By downloading an image, the
5+
QIR Alliance grants you, and you accept, a non-exclusive license to use it. All
6+
right, title and interest in the image, including the copyright therein, is
7+
retained by the QIR Alliance.

minutes/sc-meeting-2021-10.md

+28-28
Original file line numberDiff line numberDiff line change
@@ -25,39 +25,39 @@ Fabrice Franchon, Kalan Snyder, Thien Nguyen, Tom Lubinski, Pauline Jordan
2525
## Key decisions (all voted unanimously)
2626

2727
- Logo:
28-
- We are choosing Option A (as referred in attached presentation).
29-
- Pauline will investigate some variants to possibly make the “I” more
30-
obvious but we did not seem to consider this as blocking. Fabrice to share
31-
back the additional options.
28+
- We are choosing Option A (as referred in attached presentation).
29+
- Pauline will investigate some variants to possibly make the “I” more
30+
obvious but we did not seem to consider this as blocking. Fabrice to share
31+
back the additional options.
3232
- GitHub
33-
- Steering Committee will appoint maintainers of code who will have
34-
flexibility to maintain while the steering committee observes
35-
- Steering Committee members have admin rights.
36-
- There will be one repo per spec, and profiles will be sub-folders
37-
- https://github.com/qir-alliance for the organization
38-
- https://github.com/qir-alliance/qir-spec for the QIR specification
33+
- Steering Committee will appoint maintainers of code who will have
34+
flexibility to maintain while the steering committee observes
35+
- Steering Committee members have admin rights.
36+
- There will be one repo per spec, and profiles will be sub-folders
37+
- https://github.com/qir-alliance for the organization
38+
- https://github.com/qir-alliance/qir-spec for the QIR specification
3939
- Specification
40-
- We will check-in the current version of the specification
41-
- Additional changes such as callable values support and adding C APIs will
42-
be added as part of the normal contribution process
40+
- We will check-in the current version of the specification
41+
- Additional changes such as callable values support and adding C APIs will
42+
be added as part of the normal contribution process
4343
- Announcements and Press Releases
44-
- Linux Foundation will announce to its members (Project notice) the
45-
formation of the Alliance within the next 2 business days
46-
- A formal press release will be published in November based on inputs from
47-
all members.
44+
- Linux Foundation will announce to its members (Project notice) the
45+
formation of the Alliance within the next 2 business days
46+
- A formal press release will be published in November based on inputs from
47+
all members.
4848
- Chair
49-
- Steering Committee unanimously elected Bettina Heim as Chair of the QIR
50-
Alliance Steering Committee
49+
- Steering Committee unanimously elected Bettina Heim as Chair of the QIR
50+
Alliance Steering Committee
5151
- Steering Committee meetings
52-
- Monthly formal cadence augmented by ad-hoc meetings and discussions
53-
- Teams will be the main meeting and communication mechanism among steering
54-
committee members
55-
- Keep Steering Committee meetings closed until the most public announcement
56-
of the creation of QIR Alliance (right before Q2B). We will likely open-up
57-
after that, while making sure there is a proper organization to receive
58-
input from non-steering committee members.
52+
- Monthly formal cadence augmented by ad-hoc meetings and discussions
53+
- Teams will be the main meeting and communication mechanism among steering
54+
committee members
55+
- Keep Steering Committee meetings closed until the most public announcement
56+
of the creation of QIR Alliance (right before Q2B). We will likely open-up
57+
after that, while making sure there is a proper organization to receive
58+
input from non-steering committee members.
5959

60-
## Follow-ups / Action items
60+
## Follow-ups / Action items
6161

6262
- Fabrice to identify if https://github.com/qira can be obtained from their
6363
current owners.
@@ -70,4 +70,4 @@ Fabrice Franchon, Kalan Snyder, Thien Nguyen, Tom Lubinski, Pauline Jordan
7070
## Additional notes
7171

7272
- GitHub
73-
- Linux Foundation needs to be an owner (but not an administrator)
73+
- Linux Foundation needs to be an owner (but not an administrator)

minutes/sc-meeting-2021-11.md

+55-47
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ Kalan Snyder, Thien Nguyen, Tom Lubinski, Travis Humble, Seth Newberry (Guest)
99
- Proposals for a discussion and vote on steering committee attendance
1010
- Admin tasks and follow-ups status
1111
- Q2B attendance and abstract draft
12-
- 3 requests for repos under qir-alliance
13-
need formal approval and confirm level of autonomy of maintainers
12+
- 3 requests for repos under qir-alliance need formal approval and confirm level
13+
of autonomy of maintainers
1414
- QCOR (ORNL)
1515
- PyQIR (MS)
1616
- QAT (MS)
@@ -26,52 +26,60 @@ need formal approval and confirm level of autonomy of maintainers
2626

2727
- Steering Committee members can invite Subject Matter Experts, including if
2828
they are not members.
29-
- Alex McCaskey to be invited to subsequent meetings
30-
- Communication:<br/>
31-
> *"The steering committee meets once a month. The meetings are not open to the
32-
public at this time, but minutes will be published on [link]. The steering
33-
committee may invite domain experts and repository maintainers to join the
34-
discussions. Please reach out to [email] if you would like to propose a topic
35-
for consideration and/or would like to volunteer your time and expertise."*
29+
- Alex McCaskey to be invited to subsequent meetings
30+
- Communication:<br/>
31+
> *"The steering committee meets once a month. The meetings are not open to
32+
the public at this time, but minutes will be published on [link]. The
33+
steering committee may invite domain experts and repository maintainers to
34+
join the discussions. Please reach out to [email] if you would like to
35+
propose a topic for consideration and/or would like to volunteer your time
36+
and expertise."*
3637
- Repos maintainers can attend steering committee meetings
3738
- Q2B presentation
3839
- Title and abstract: <br/>
39-
>***Empowering heterogeneous quantum computing of today and tomorrow with QIR**<br/>
40+
>***Empowering heterogeneous quantum computing of today and tomorrow with
41+
>QIR**<br/>
4042
Quantum computing systems and languages evolve at a fast pace, promising
4143
exciting advances in how classical and quantum computations can interact at
4244
the hardware level. To provide interoperability between different frameworks
4345
and accelerate progress for all parties, it is imperative for the ecosystem
4446
to develop and share a forward-looking quantum intermediate representation
4547
(QIR) that leverages state of the art compiler technology capable of
4648
supporting the full variety of heterogenous quantum processors. <br/>
47-
The QIR Alliance is a joint effort to develop an intermediate representation that
48-
enables full interoperability within the quantum ecosystem, and provides a
49-
holistic program representation suitable for taking full advantage of the
50-
unique and distinct capabilities of present and future quantum processors. <br/>
51-
In this session we will explore critical advances based on three examples that
52-
illustrate the novel opportunities for a spectrum of use cases arising from this effort.”*
49+
The QIR Alliance is a joint effort to develop an intermediate representation
50+
that enables full interoperability within the quantum ecosystem, and
51+
provides a holistic program representation suitable for taking full
52+
advantage of the unique and distinct capabilities of present and future
53+
quantum processors. <br/>
54+
In this session we will explore critical advances based on three examples
55+
that illustrate the novel opportunities for a spectrum of use cases arising
56+
from this effort.”*
5357
- Mode of operation and communication of code repositories
54-
- Addition of the following repos: PyQIR, QAT, QCOR
55-
- Requirements for code repositories:
56-
>*"Anyone can ask to add an open-source repository under the QIR Alliance,
57-
and the steering committee will approve or deny the request depending on
58-
whether the minimum requirements as outlined below are satisfied. The
59-
steering committee may also solicit such contributions. <br/>
60-
> For code repositories under the QIR Alliance,
61-
> - At least two maintainers need to be designated for the repository,
62-
and a way to contact the maintainers should be shared with the
63-
steering committee.
64-
> - The repository is either actively maintained by the maintainers or the
65-
repository is archived.
66-
> - The code of conduct aligns in spirit with the code of conduct of the
67-
broader org.
68-
> - The project should be related in content to the scope defined for the
69-
QIR Alliance.
70-
> - All code should be licensed under MIT license, and
71-
contributors to the repository must accept a CLA defined by the maintainers.
72-
<br/><br/>
73-
74-
> The QIR Alliance will reach out to the maintainers if a repository no longer satisfies these criteria to reach an agreement and resolve the situation. If the maintainers cannot be reached or are unable or unwilling to take action, the QIR Alliance reserves the right to archive the repository."*
58+
- Addition of the following repos: PyQIR, QAT, QCOR
59+
- Requirements for code repositories:
60+
>*"Anyone can ask to add an open-source repository under the QIR Alliance,
61+
and the steering committee will approve or deny the request depending on
62+
whether the minimum requirements as outlined below are satisfied. The
63+
steering committee may also solicit such contributions.* <br/>
64+
>*For code repositories under the QIR Alliance,*
65+
>
66+
> - *At least two maintainers need to be designated for the repository, and
67+
a way to contact the maintainers should be shared with the steering
68+
committee.*
69+
> - *The repository is either actively maintained by the maintainers or the
70+
repository is archived.*
71+
> - *The code of conduct aligns in spirit with the code of conduct of the
72+
broader org.*
73+
> - *The project should be related in content to the scope defined for the
74+
QIR Alliance.*
75+
> - *All code should be licensed under MIT license, and contributors to the
76+
repository must accept a CLA defined by the maintainers.* <br/><br/>
77+
>
78+
> *The QIR Alliance will reach out to the maintainers if a repository no
79+
longer satisfies these criteria to reach an agreement and resolve the
80+
situation. If the maintainers cannot be reached or are unable or unwilling
81+
to take action, the QIR Alliance reserves the right to archive the
82+
repository."*
7583

7684
## Additional notes
7785

@@ -83,17 +91,17 @@ need formal approval and confirm level of autonomy of maintainers
8391
- JDF needs a FAQ that they can use to help answer questions from parties
8492
interested in QIR Alliance (some Platinum members already expressed interest).
8593
- Attending Q2B:
86-
- Honeywell: Pete Campora
87-
- Microsoft: Bettina Heim, Chris Granada, Fabrice Frachon, Viktor Veis
88-
- ORNL: Not attending
89-
- QCI: Tom Lubinski
90-
- Rigetti: Kalan Snyder
94+
- Honeywell: Pete Campora
95+
- Microsoft: Bettina Heim, Chris Granada, Fabrice Frachon, Viktor Veis
96+
- ORNL: Not attending
97+
- QCI: Tom Lubinski
98+
- Rigetti: Kalan Snyder
9199
- Adding external code:
92-
- Do not accept anything that contain something like a GPL license as it
93-
relicenses the entire code.
94-
- Should be ok to add copyrighted materials (Seth following-up)
100+
- Do not accept anything that contain something like a GPL license as it
101+
relicenses the entire code.
102+
- Should be ok to add copyrighted materials (Seth following-up)
95103
- Press release
96-
- Members will be amplifying on their social channels but no specific blog
97-
planned.
104+
- Members will be amplifying on their social channels but no specific blog
105+
planned.
98106
- We should create an Info@ and legal@ alias to direct interest (Fabrice to
99107
investigate best option).

workflow-templates/utils/md_lint_config.yml

+5-1
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,8 @@ no-duplicate-heading:
2222

2323
# Allow selective inline markdown
2424
no-inline-html:
25-
allowed_elements: [br]
25+
allowed_elements: [br]
26+
27+
# Allow exclamation mark and question marks in headings
28+
MD026:
29+
punctuation: ".,;:。,;:!"

0 commit comments

Comments
 (0)