Skip to content

Commit bdd23aa

Browse files
authored
Merge pull request #3048 from metacpan/oalders/precious
oalders/precious
2 parents 4b4afe7 + f81d1f4 commit bdd23aa

28 files changed

+446
-347
lines changed

.github/CONTRIBUTING.md

Lines changed: 60 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -6,40 +6,54 @@ We are always after more contributors and suggestions.
66

77
#### Does it relate to our API (backend)... ?
88

9-
1. Please check the [previously reported API issues](https://github.com/CPAN-API/cpan-api/issues)
10-
2. Please check the [Wishlist](https://github.com/CPAN-API/cpan-api/wiki/Wishlist). If you can't find it already there:
11-
* If it's a wishlist idea, please edit the [wiki](https://github.com/CPAN-API/cpan-api/wiki/Wishlist) (add a 'wishlist_MYIDEA' page if you need more space!)
12-
* If it's an actual bug [create a new issue](https://github.com/CPAN-API/cpan-api/issues/new)
9+
1. Please check the
10+
[previously reported API issues](https://github.com/CPAN-API/cpan-api/issues)
11+
2. Please check the
12+
[Wishlist](https://github.com/CPAN-API/cpan-api/wiki/Wishlist). If you can't
13+
find it already there:
14+
- If it's a wishlist idea, please edit the
15+
[wiki](https://github.com/CPAN-API/cpan-api/wiki/Wishlist) (add a
16+
'wishlist_MYIDEA' page if you need more space!)
17+
- If it's an actual bug
18+
[create a new issue](https://github.com/CPAN-API/cpan-api/issues/new)
1319

1420
#### If you are not sure, or it is related to https://metacpan.org/ front end:
1521

16-
1. Please check the [previously reported Web issues](https://github.com/CPAN-API/metacpan-web/issues)
17-
2. Please check the [Wishlist](https://github.com/CPAN-API/cpan-api/wiki/Wishlist). If you can't find it already there:
18-
* If it's a wishlist idea, please edit the [wiki](https://github.com/CPAN-API/cpan-api/wiki/Wishlist) (add a 'wishlist_MYIDEA' page if you need more space!)
19-
* If it's an actual bug [create a new issue](https://github.com/CPAN-API/metacpan-web/issues/new)
22+
1. Please check the
23+
[previously reported Web issues](https://github.com/CPAN-API/metacpan-web/issues)
24+
2. Please check the
25+
[Wishlist](https://github.com/CPAN-API/cpan-api/wiki/Wishlist). If you can't
26+
find it already there:
27+
- If it's a wishlist idea, please edit the
28+
[wiki](https://github.com/CPAN-API/cpan-api/wiki/Wishlist) (add a
29+
'wishlist_MYIDEA' page if you need more space!)
30+
- If it's an actual bug
31+
[create a new issue](https://github.com/CPAN-API/metacpan-web/issues/new)
2032

2133
## Contributing code
2234

2335
Come talk to us on IRC (see below), or send a pull request and we'll respond
24-
there. If you implement a new feature, please add a note about it to the
25-
News.md file at the top level of metacpan-web so that it will appear in our
26-
news feed.
36+
there. If you implement a new feature, please add a note about it to the News.md
37+
file at the top level of metacpan-web so that it will appear in our news feed.
2738

28-
If you aren't using the VM, remember to enable the pre-commit hook before you start working.
39+
If you aren't using the VM, remember to enable the pre-commit hook before you
40+
start working.
2941

3042
sh git/setup.sh
3143

3244
These links will get you going quickly:
3345

34-
* [Using our developer VM](https://github.com/CPAN-API/metacpan-developer) to get you going in minutes (depending on bandwidth)
35-
* [Front end bug list](https://github.com/CPAN-API/metacpan-web/issues)
36-
* [API (back end) bug list](https://github.com/CPAN-API/cpan-api/issues)
37-
* [Wishlist](https://github.com/CPAN-API/cpan-api/wiki/Wishlist) - things that probably need doing
46+
- [Using our developer VM](https://github.com/CPAN-API/metacpan-developer) to
47+
get you going in minutes (depending on bandwidth)
48+
- [Front end bug list](https://github.com/CPAN-API/metacpan-web/issues)
49+
- [API (back end) bug list](https://github.com/CPAN-API/cpan-api/issues)
50+
- [Wishlist](https://github.com/CPAN-API/cpan-api/wiki/Wishlist) - things that
51+
probably need doing
3852

3953
# Git workflow
4054

4155
We try to keep a clean git history, so if it all possible, please rebase to get
42-
the latest changes from master _before_ submitting a pull request. You'll only
56+
the latest changes from master _before_ submitting a pull request. You'll only
4357
need to do the first command (git remote add) once in your local checkout.
4458

4559
git remote add upstream https://github.com/CPAN-API/metacpan-web.git
@@ -50,17 +64,18 @@ which are no longer relevant to your branch before submitting your work.
5064

5165
git rebase -i master
5266

53-
If you are not comfortable with rebasing, but want to use it, check out the steps
54-
from [here](https://help.github.com/articles/using-git-rebase/).
67+
If you are not comfortable with rebasing, but want to use it, check out the
68+
steps from [here](https://help.github.com/articles/using-git-rebase/).
5569

5670
# Coding conventions
5771

58-
Please try to follow the conventions already been used in the code base. This
59-
will generally be the right thing to do. Our standards are improving, so even
60-
if you do follow what you see, we may ask you to make some changes, but that is
61-
a good thing. We are trying to keep things tidy.
72+
Please try to follow the conventions already been used in the code base. This
73+
will generally be the right thing to do. Our standards are improving, so even if
74+
you do follow what you see, we may ask you to make some changes, but that is a
75+
good thing. We are trying to keep things tidy.
6276

63-
If you are using the [developer VM](https://github.com/CPAN-API/metacpan-developer) you can run:
77+
If you are using the
78+
[developer VM](https://github.com/CPAN-API/metacpan-developer) you can run:
6479

6580
```sh
6681
/home/vagrant/carton/metacpan-web/bin/tidyall
@@ -69,71 +84,75 @@ If you are using the [developer VM](https://github.com/CPAN-API/metacpan-develop
6984
## Perl Best Practices
7085

7186
In general, the concepts discussed in "Perl Best Practices" are a good starting
72-
point. Use autodie where possible and MetaCPAN::Web::Types when creating new
73-
Moose attributes. Many of the other standards will be enforced by Perl::Critic.
87+
point. Use autodie where possible and MetaCPAN::Web::Types when creating new
88+
Moose attributes. Many of the other standards will be enforced by Perl::Critic.
7489

7590
## Clear > Concise
7691

7792
Take pains to use variable names which are easy to understand and to write
78-
readable code. We value readable code over concise code. Use singular nouns
79-
for class names. Use verbs for method names.
93+
readable code. We value readable code over concise code. Use singular nouns for
94+
class names. Use verbs for method names.
8095

8196
## Try::Tiny > eval { ... }
8297

83-
You will see many eval statements in the code. We would like to standardize on
98+
You will see many eval statements in the code. We would like to standardize on
8499
Try::Tiny, so feel free to swap out any eval with a Try::Tiny and use Try::Tiny
85100
in all new code.
86101

87102
## Prefer single quotes
88103

89-
Always use single quotes in cases where there is no variable interpolation. If
104+
Always use single quotes in cases where there is no variable interpolation. If
90105
there is a single quote in the quoted item, use curly quotes.
91106

92107
q{Isn't this a lovely day};
93108

94109
## Include a test (or more!)
95110

96111
Any time when a pull request includes a test, it makes it easier for us to
97-
review and accept, so please do test your changes whenever possible. If your
112+
review and accept, so please do test your changes whenever possible. If your
98113
pull request includes visual changes, please include a before and after screen
99114
shot, so that we can better understand the problem you're trying to solve.
100115

101116
## Dependencies
102117

103118
Introducing new dependencies is fine, if they solve a specific problem which
104-
current dependencies cannot address. If we prefer a different module to be used,
119+
current dependencies cannot address. If we prefer a different module to be used,
105120
we'll let you know.
106121

107122
## It's OK to be controversial
108123

109124
If a pull request contains any controversial changes, we'll likely wait for some
110-
feedback from several developers before a merge. If you think your changes may
125+
feedback from several developers before a merge. If you think your changes may
111126
be controversial, feel free to discuss them in a GitHub issue before starting to
112127
write any code.
113128

114129
## Travis is your friend
115130

116-
We use Travis to test all code changes. After submitting your pull request,
131+
We use Travis to test all code changes. After submitting your pull request,
117132
remember to check back to see whether Travis has come back with any test
118-
failures. We do get some false negatives. If your pull request failed for
133+
failures. We do get some false negatives. If your pull request failed for
119134
reasons unrelated to your changes, we may still be able to merge your work.
120135

121136
# Additional Resources
122137

123-
* [\#metacpan](http://widget01.mibbit.com/?autoConnect=true&server=irc.perl.org&channel=%23metacpan&nick=) IRC channel on irc.perl.org
138+
- [\#metacpan](http://widget01.mibbit.com/?autoConnect=true&server=irc.perl.org&channel=%23metacpan&nick=)
139+
IRC channel on irc.perl.org
124140

125141
# Current Policies
126142

127143
### What is indexed?
128144

129-
* Perl distributions which contain Perl packages.
145+
- Perl distributions which contain Perl packages.
130146

131147
### When are issues closed?
132148

133149
We want to keep the issue list manageable, so we can focus on what actually
134-
needs fixing. If you feel an issue needs opening again, please add a comment
150+
needs fixing. If you feel an issue needs opening again, please add a comment
135151
explaining why it needs re-opening and we'll look at it again.
136152

137-
* Issues will be closed and moved to [Wishlist](https://github.com/CPAN-API/cpan-api/wiki/Wishlist) if they are not actual bugs
138-
* Issues we think we have addressed will be closed
139-
* Issues we are not going to take any further action on without more information will be closed
153+
- Issues will be closed and moved to
154+
[Wishlist](https://github.com/CPAN-API/cpan-api/wiki/Wishlist) if they are not
155+
actual bugs
156+
- Issues we think we have addressed will be closed
157+
- Issues we are not going to take any further action on without more information
158+
will be closed
Lines changed: 22 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,42 +1,43 @@
11
---
22
name: Bug report
33
about: Create a report to help us improve
4-
title: ''
4+
title: ""
55
labels: ["type:Bug"]
6-
assignees: ''
7-
6+
assignees: ""
87
---
98

109
**PLEASE NOTE**
1110

12-
This issue tracker is for bugs in the https://metacpan.org web site. If you are are reporting a bug about anything else, this is most likely not the correct venue. Please use the left side menu for the module you are interested in and click the `Issues` link in order to find the bug tracker for your module.
11+
This issue tracker is for bugs in the https://metacpan.org web site. If you are
12+
are reporting a bug about anything else, this is most likely not the correct
13+
venue. Please use the left side menu for the module you are interested in and
14+
click the `Issues` link in order to find the bug tracker for your module.
15+
16+
**Describe the bug** A clear and concise description of what the bug is.
1317

14-
**Describe the bug**
15-
A clear and concise description of what the bug is.
18+
**To Reproduce** Steps to reproduce the behavior:
1619

17-
**To Reproduce**
18-
Steps to reproduce the behavior:
1920
1. Go to '...'
2021
2. Click on '....'
2122
3. Scroll down to '....'
2223
4. See error
2324

24-
**Expected behavior**
25-
A clear and concise description of what you expected to happen.
25+
**Expected behavior** A clear and concise description of what you expected to
26+
happen.
2627

27-
**Screenshots**
28-
If applicable, add screenshots to help explain your problem.
28+
**Screenshots** If applicable, add screenshots to help explain your problem.
2929

3030
**Desktop (please complete the following information):**
31-
- OS: [e.g. iOS]
32-
- Browser [e.g. chrome, safari]
33-
- Version [e.g. 22]
31+
32+
- OS: [e.g. iOS]
33+
- Browser [e.g. chrome, safari]
34+
- Version [e.g. 22]
3435

3536
**Smartphone (please complete the following information):**
36-
- Device: [e.g. iPhone6]
37-
- OS: [e.g. iOS8.1]
38-
- Browser [e.g. stock browser, safari]
39-
- Version [e.g. 22]
4037

41-
**Additional context**
42-
Add any other context about the problem here.
38+
- Device: [e.g. iPhone6]
39+
- OS: [e.g. iOS8.1]
40+
- Browser [e.g. stock browser, safari]
41+
- Version [e.g. 22]
42+
43+
**Additional context** Add any other context about the problem here.
Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,18 @@
11
---
22
name: Feature request
33
about: Request a feature to be added
4-
title: ''
4+
title: ""
55
labels: ["type:Feature"]
6-
assignees: ''
7-
6+
assignees: ""
87
---
9-
**Describe the feature**
10-
A clear and concise description of what the feature is.
8+
9+
**Describe the feature** A clear and concise description of what the feature is.
1110

1211
**Which problem does it solve?**
1312

14-
**Expected behavior**
15-
A clear and concise description of what you expect to see.
13+
**Expected behavior** A clear and concise description of what you expect to see.
1614

17-
**Volunteering**
18-
Are you willing to implement (or help to implement) this feature?
15+
**Volunteering** Are you willing to implement (or help to implement) this
16+
feature?
1917

20-
**Additional context**
21-
Add any other context about the request here.
18+
**Additional context** Add any other context about the request here.
Lines changed: 33 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: "Code scanning - action"
1+
name: 'Code scanning - action'
22

33
on:
44
push:
@@ -10,39 +10,38 @@ on:
1010

1111
jobs:
1212
CodeQL-Build:
13-
1413
runs-on: ubuntu-22.04
1514

1615
steps:
17-
- name: Checkout repository
18-
uses: actions/checkout@v4
19-
with:
20-
# We must fetch at least the immediate parents so that if this is
21-
# a pull request then we can checkout the head.
22-
fetch-depth: 2
23-
24-
# Initializes the CodeQL tools for scanning.
25-
- name: Initialize CodeQL
26-
uses: github/codeql-action/init@v3
27-
# Override language selection by uncommenting this and choosing your languages
28-
# with:
29-
# languages: go, javascript, csharp, python, cpp, java
30-
31-
# Autobuild attempts to build any compiled languages (C/C++, C#, or Java).
32-
# If this step fails, then you should remove it and run the build manually (see below)
33-
- name: Autobuild
34-
uses: github/codeql-action/autobuild@v3
35-
36-
# ℹ️ Command-line programs to run using the OS shell.
37-
# 📚 https://git.io/JvXDl
38-
39-
# ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
40-
# and modify them (or add more) to build your code if your project
41-
# uses a compiled language
42-
43-
#- run: |
44-
# make bootstrap
45-
# make release
46-
47-
- name: Perform CodeQL Analysis
48-
uses: github/codeql-action/analyze@v3
16+
- name: Checkout repository
17+
uses: actions/checkout@v4
18+
with:
19+
# We must fetch at least the immediate parents so that if this is
20+
# a pull request then we can checkout the head.
21+
fetch-depth: 2
22+
23+
# Initializes the CodeQL tools for scanning.
24+
- name: Initialize CodeQL
25+
uses: github/codeql-action/init@v3
26+
# Override language selection by uncommenting this and choosing your languages
27+
# with:
28+
# languages: go, javascript, csharp, python, cpp, java
29+
30+
# Autobuild attempts to build any compiled languages (C/C++, C#, or Java).
31+
# If this step fails, then you should remove it and run the build manually (see below)
32+
- name: Autobuild
33+
uses: github/codeql-action/autobuild@v3
34+
35+
# ℹ️ Command-line programs to run using the OS shell.
36+
# 📚 https://git.io/JvXDl
37+
38+
# ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
39+
# and modify them (or add more) to build your code if your project
40+
# uses a compiled language
41+
42+
#- run: |
43+
# make bootstrap
44+
# make release
45+
46+
- name: Perform CodeQL Analysis
47+
uses: github/codeql-action/analyze@v3

0 commit comments

Comments
 (0)