Skip to content

Commit bbc51e5

Browse files
authored
Merge pull request #590 from sbillinge/jan25
added pokratath etc.. Also prettify citations
2 parents a2929ee + 3a07709 commit bbc51e5

24 files changed

+1650
-1429
lines changed

.codespell/ignore_lines.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
;; Please include filenames and explanations for each ignored line.
2+
;; See https://docs.openverse.org/meta/codespell.html for docs.

.codespell/ignore_words.txt

Lines changed: 55 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,55 @@
1+
;; Please include explanations for each ignored word (lowercase).
2+
;; See https://docs.openverse.org/meta/codespell.html for docs.
3+
4+
;; abbreviation for "materials" often used in a journal title
5+
mater
6+
7+
;; alternative use of socioeconomic
8+
socio-economic
9+
10+
;; Frobenius norm used in np.linalg.norm
11+
fro
12+
13+
;; The facility not the attitude
14+
SLAC
15+
16+
;; The chemicals
17+
Nd
18+
ThSe
19+
Te
20+
NWO
21+
AFE
22+
23+
;; The names of collaborators
24+
Nam
25+
Skelton
26+
Tennant
27+
BASF
28+
Joo
29+
Hart
30+
Comin
31+
Stach
32+
Comor
33+
Flor
34+
Winn
35+
Studi
36+
37+
;; Whet as in appetite
38+
whet
39+
40+
;; journal abbreviations
41+
Commun
42+
Ser
43+
bridg
44+
slac
45+
SHS
46+
stach
47+
selt
48+
als
49+
basf
50+
51+
;; citation things
52+
annote
53+
54+
;; the state
55+
ND

.flake8

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
# As of now, flake8 does not natively support configuration via pyproject.toml
2+
# https://github.com/microsoft/vscode-flake8/issues/135
3+
[flake8]
4+
exclude =
5+
.git,
6+
__pycache__,
7+
build,
8+
dist,
9+
doc/source/conf.py
10+
max-line-length = 115
11+
# Ignore some style 'errors' produced while formatting by 'black'
12+
# https://black.readthedocs.io/en/stable/guides/using_black_with_other_tools.html#labels-why-pycodestyle-warnings
13+
extend-ignore = E203

.github/workflows/publish.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,18 @@
1-
name: Publish Group Webiste
1+
name: Publish Group Website
22

33
on:
44
push:
55
branches:
66
- main
77
workflow_dispatch:
88

9-
109
jobs:
1110
publish:
1211
name: Miniconda ${{ matrix.os }}
1312
runs-on: ${{ matrix.os }}
1413
strategy:
15-
matrix:
16-
os: ["ubuntu-latest"]
14+
matrix:
15+
os: ['ubuntu-latest']
1716
steps:
1817
- name: check out rg-db-public
1918
uses: actions/checkout@v3

.pre-commit-config.yaml

Lines changed: 70 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,70 @@
1+
default_language_version:
2+
python: python3
3+
ci:
4+
autofix_commit_msg: |
5+
[pre-commit.ci] auto fixes from pre-commit hooks
6+
autofix_prs: true
7+
autoupdate_branch: "pre-commit-autoupdate"
8+
autoupdate_commit_msg: "[pre-commit.ci] pre-commit autoupdate"
9+
autoupdate_schedule: monthly
10+
skip: [no-commit-to-branch]
11+
submodules: false
12+
repos:
13+
- repo: https://github.com/pre-commit/pre-commit-hooks
14+
rev: v4.6.0
15+
hooks:
16+
- id: check-yaml
17+
exclude: '^{{ cookiecutter.github_repo_name }}/.*\.yml$' # {{ cookiecutter.github_repo_name }} contains jinja variables in .yml, hook fails
18+
- id: end-of-file-fixer
19+
- id: trailing-whitespace
20+
- id: check-case-conflict
21+
- id: check-merge-conflict
22+
- id: check-toml
23+
exclude: '^{{ cookiecutter.github_repo_name }}/.*\.toml$' # {{ cookiecutter.github_repo_name }} contains jinja variables in pyproject.toml, hook fails
24+
- id: check-added-large-files
25+
- repo: https://github.com/psf/black
26+
rev: 24.4.2
27+
hooks:
28+
- id: black
29+
exclude: '^({{ cookiecutter.github_repo_name }}/.*\.py|hooks/post_gen_project\.py)$' # {{ cookiecutter.github_repo_name }} contains jinja variables in .py, hook fails. hooks/post_gen_project.py contains jinja variables of # (c) {% now 'utc', '%Y' %} which is not a valid black format
30+
- repo: https://github.com/pycqa/flake8
31+
rev: 7.0.0
32+
hooks:
33+
- id: flake8
34+
exclude: '^({{ cookiecutter.github_repo_name }}/.*\.py|hooks/post_gen_project\.py)$' # Same comment as black above
35+
- repo: https://github.com/pycqa/isort
36+
rev: 5.13.2
37+
hooks:
38+
- id: isort
39+
args: ["--profile", "black"]
40+
- repo: https://github.com/kynan/nbstripout
41+
rev: 0.7.1
42+
hooks:
43+
- id: nbstripout
44+
- repo: https://github.com/pre-commit/pre-commit-hooks
45+
rev: v4.4.0
46+
hooks:
47+
- id: no-commit-to-branch
48+
name: Prevent Commit to Main Branch
49+
args: ["--branch", "main"]
50+
stages: [pre-commit]
51+
- repo: https://github.com/codespell-project/codespell
52+
rev: v2.3.0
53+
hooks:
54+
- id: codespell
55+
additional_dependencies:
56+
- tomli
57+
# prettier - multi formatter for .json, .yml, and .md files
58+
- repo: https://github.com/pre-commit/mirrors-prettier
59+
rev: f12edd9c7be1c20cfa42420fd0e6df71e42b51ea # frozen: v4.0.0-alpha.8
60+
hooks:
61+
- id: prettier
62+
additional_dependencies:
63+
- "prettier@^3.2.4"
64+
# docformatter - PEP 257 compliant docstring formatter
65+
- repo: https://github.com/s-weigand/docformatter
66+
rev: 5757c5190d95e5449f102ace83df92e7d3b06c6c
67+
hooks:
68+
- id: docformatter
69+
additional_dependencies: [tomli]
70+
args: [--in-place, --config, ./pyproject.toml]

.prettierignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
# Ignore jinja2 variables within the project
2+
{{ cookiecutter.github_repo_name }}

.prettierrc

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
{
2+
"overrides": [
3+
{
4+
"files": "*.yml",
5+
"options": {
6+
"singleQuote": true,
7+
"proseWrap": "preserve"
8+
}
9+
}
10+
],
11+
"printWidth": 115
12+
}

LICENSE

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,4 +21,3 @@ SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
2121
CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
2222
OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
2323
OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
24-

README.md

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,26 @@
11
![CI](https://github.com/billingegroup/rg-db-public/actions/workflows/main.yml/badge.svg)
2+
23
# rg-db-public
3-
Billinge Group Public Research Group Database
44

5+
Billinge Group Public Research Group Database
56

67
This repo contains public facing data from the Billinge Group in the Applied
7-
Physics and Applied Mathematics department at Columbia University. It will be used
8+
Physics and Applied Mathematics department at Columbia University. It will be used
89
to build the group dynamic web-site at `https://billingegroup.github.io` using
9-
regolith (https://github.com/regro/regolith). Regolith will also build
10+
regolith (https://github.com/regro/regolith). Regolith will also build
1011
people's public facing cv's and publication lists from this information here.
1112

1213
You need to be a current or former member of the Billinge Group at Columbia
13-
University to update entries in this database. Please reach out to Prof.
14+
University to update entries in this database. Please reach out to Prof.
1415
Billinge (sb2896@columbia.edu) if you are interested in joining the group.
1516

16-
For group members, the current branching workflow is to directly clone this
17+
For group members, the current branching workflow is to directly clone this
1718
repo to your local, make pushes to branches directly on this GitHub repository
18-
and create a PR into the default branch `main`. We are no longer using a
19+
and create a PR into the default branch `main`. We are no longer using a
1920
forking workflow (i.e., don't create a fork).
2021

2122
These commands might help you get going:
23+
2224
```
2325
cd dbs
2426
git clone git@github.com:Billingegroup/rg-db-public.git
@@ -27,28 +29,31 @@ git add <files>
2729
git commit -m "an informative commit message"
2830
git push -u origin <branch_name>
2931
```
30-
then go to GitHub to open a PR into `main`, so it will look like
32+
33+
then go to GitHub to open a PR into `main`, so it will look like
3134
`base:main <- compare:<branch_name>`
3235

3336
Do not ever merge directly into main (actually you shouldn't be able to on GitHub).
34-
This merge will be done by Prof. Billinge after review.
37+
This merge will be done by Prof. Billinge after review.
3538

3639
To prevent nasty accidents,
37-
1. create a file in `rg-db-public/.git/hooks` called `pre-commit` and copy-paste the
38-
following into it and save it
40+
41+
1. create a file in `rg-db-public/.git/hooks` called `pre-commit` and copy-paste the
42+
following into it and save it
3943
2. Mark the new file as executable: `chmod +x rg-db-public/.git/hooks/pre-commit`.
4044

41-
This will prevent you accidentally making local changes to your main branch that won't
45+
This will prevent you accidentally making local changes to your main branch that won't
4246
be mergeable.
47+
4348
```
4449
#!/bin/sh
4550
#
4651
# To prevent accidental commits to main,
47-
# copy this file, with name
48-
# "pre-commit"
49-
# to the
52+
# copy this file, with name
53+
# "pre-commit"
54+
# to the
5055
# .git/hooks (notice the dot in front of git)
51-
# directory in this
56+
# directory in this
5257
# repo
5358
5459
branch="$(git rev-parse --abbrev-ref HEAD)"

0 commit comments

Comments
 (0)