Skip to content

chore: clean duplicated file - #94

Merged
sbillinge merged 2 commits into
diffpy:mainfrom
stevenhua0320:clean-file
Apr 7, 2026
Merged

chore: clean duplicated file#94
sbillinge merged 2 commits into
diffpy:mainfrom
stevenhua0320:clean-file

Conversation

@stevenhua0320

@stevenhua0320stevenhua0320 commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

@sbillinge ready to review. We have already had one CODE-OF-CONDUCT.rst by rescikit-package, so we would like to remove the duplicated one. Also I checked the tree structure of current repo, and we are all good with the naming convention. See the tree below.
Closes#88
Closes#78

(diffpy.fourigui-env) huarundong@Stevens-MacBook-Pro-405 diffpy.fourigui % tree
.
├── AUTHORS.rst
├── CHANGELOG.rst
├── CODE-OF-CONDUCT.rst
├── LICENSE.rst
├── MANIFEST.in
├── README.rst
├── cookiecutter.json
├── dist
│ ├── diffpy_fourigui-0.0.1-py3-none-any.whl
│ └── diffpy_fourigui-0.0.1.tar.gz
├── docs
│ ├── Makefile
│ ├── build
│ │ ├── doctrees
│ │ │ ├── api
│ │ │ │ └── diffpy.fourigui.doctree
│ │ │ ├── environment.pickle
│ │ │ ├── index.doctree
│ │ │ ├── license.doctree
│ │ │ ├── release.doctree
│ │ │ └── tutorial.doctree
│ │ └── html
│ │ ├── _modules
│ │ │ ├── diffpy
│ │ │ │ └── fourigui
│ │ │ │ └── fourigui.html
│ │ │ └── index.html
│ │ ├── _sources
│ │ │ ├── api
│ │ │ │ └── diffpy.fourigui.rst.txt
│ │ │ ├── index.rst.txt
│ │ │ ├── license.rst.txt
│ │ │ ├── release.rst.txt
│ │ │ └── tutorial.rst.txt
│ │ ├── _static
│ │ │ ├── _sphinx_javascript_frameworks_compat.js
│ │ │ ├── base-stemmer.js
│ │ │ ├── basic.css
│ │ │ ├── check-solid.svg
│ │ │ ├── clipboard.min.js
│ │ │ ├── copy-button.svg
│ │ │ ├── copybutton.css
│ │ │ ├── copybutton.js
│ │ │ ├── copybutton_funcs.js
│ │ │ ├── css
│ │ │ │ ├── badge_only.css
│ │ │ │ ├── fonts
│ │ │ │ │ ├── Roboto-Slab-Bold.woff
│ │ │ │ │ ├── Roboto-Slab-Bold.woff2
│ │ │ │ │ ├── Roboto-Slab-Regular.woff
│ │ │ │ │ ├── Roboto-Slab-Regular.woff2
│ │ │ │ │ ├── fontawesome-webfont.eot
│ │ │ │ │ ├── fontawesome-webfont.svg
│ │ │ │ │ ├── fontawesome-webfont.ttf
│ │ │ │ │ ├── fontawesome-webfont.woff
│ │ │ │ │ ├── fontawesome-webfont.woff2
│ │ │ │ │ ├── lato-bold-italic.woff
│ │ │ │ │ ├── lato-bold-italic.woff2
│ │ │ │ │ ├── lato-bold.woff
│ │ │ │ │ ├── lato-bold.woff2
│ │ │ │ │ ├── lato-normal-italic.woff
│ │ │ │ │ ├── lato-normal-italic.woff2
│ │ │ │ │ ├── lato-normal.woff
│ │ │ │ │ └── lato-normal.woff2
│ │ │ │ └── theme.css
│ │ │ ├── doctools.js
│ │ │ ├── documentation_options.js
│ │ │ ├── english-stemmer.js
│ │ │ ├── file.png
│ │ │ ├── fonts
│ │ │ │ ├── Lato
│ │ │ │ │ ├── lato-bold.eot
│ │ │ │ │ ├── lato-bold.ttf
│ │ │ │ │ ├── lato-bold.woff
│ │ │ │ │ ├── lato-bold.woff2
│ │ │ │ │ ├── lato-bolditalic.eot
│ │ │ │ │ ├── lato-bolditalic.ttf
│ │ │ │ │ ├── lato-bolditalic.woff
│ │ │ │ │ ├── lato-bolditalic.woff2
│ │ │ │ │ ├── lato-italic.eot
│ │ │ │ │ ├── lato-italic.ttf
│ │ │ │ │ ├── lato-italic.woff
│ │ │ │ │ ├── lato-italic.woff2
│ │ │ │ │ ├── lato-regular.eot
│ │ │ │ │ ├── lato-regular.ttf
│ │ │ │ │ ├── lato-regular.woff
│ │ │ │ │ └── lato-regular.woff2
│ │ │ │ └── RobotoSlab
│ │ │ │ ├── roboto-slab-v7-bold.eot
│ │ │ │ ├── roboto-slab-v7-bold.ttf
│ │ │ │ ├── roboto-slab-v7-bold.woff
│ │ │ │ ├── roboto-slab-v7-bold.woff2
│ │ │ │ ├── roboto-slab-v7-regular.eot
│ │ │ │ ├── roboto-slab-v7-regular.ttf
│ │ │ │ ├── roboto-slab-v7-regular.woff
│ │ │ │ └── roboto-slab-v7-regular.woff2
│ │ │ ├── jquery.js
│ │ │ ├── js
│ │ │ │ ├── badge_only.js
│ │ │ │ ├── theme.js
│ │ │ │ └── versions.js
│ │ │ ├── language_data.js
│ │ │ ├── minus.png
│ │ │ ├── plus.png
│ │ │ ├── pygments.css
│ │ │ ├── searchtools.js
│ │ │ └── sphinx_highlight.js
│ │ ├── api
│ │ │ └── diffpy.fourigui.html
│ │ ├── genindex.html
│ │ ├── index.html
│ │ ├── license.html
│ │ ├── objects.inv
│ │ ├── py-modindex.html
│ │ ├── release.html
│ │ ├── search.html
│ │ ├── searchindex.js
│ │ └── tutorial.html
│ ├── make.bat
│ └── source
│ ├── _static
│ ├── api
│ │ └── diffpy.fourigui.rst
│ ├── conf.py
│ ├── img
│ ├── index.rst
│ ├── license.rst
│ ├── release.rst
│ ├── snippets
│ └── tutorial.rst
├── news
│ ├── TEMPLATE.rst
│ ├── clean-file.rst
│ ├── copyright-styling.rst
│ ├── doc-build.rst
│ ├── doc.rst
│ ├── h5py.rst
│ ├── latest-checklist.rst
│ ├── license.rst
│ ├── python-3.14.rst
│ ├── skpkg.rst
│ └── workflows.rst
├── pyproject.toml
├── requirements
│ ├── conda.txt
│ ├── docs.txt
│ ├── pip.txt
│ └── tests.txt
├── src
│ ├── diffpy
│ │ ├── __init__.py
│ │ ├── __pycache__
│ │ │ └── __init__.cpython-314.pyc
│ │ └── fourigui
│ │ ├── __init__.py
│ │ ├── __pycache__
│ │ │ ├── __init__.cpython-314.pyc
│ │ │ ├── fourigui.cpython-314.pyc
│ │ │ └── version.cpython-314.pyc
│ │ ├── fourigui.py
│ │ ├── fourigui_app.py
│ │ └── version.py
│ └── diffpy.fourigui.egg-info
│ ├── PKG-INFO
│ ├── SOURCES.txt
│ ├── dependency_links.txt
│ ├── entry_points.txt
│ ├── requires.txt
│ └── top_level.txt
└── tests
├── __pycache__
│ ├── conftest.cpython-314-pytest-9.0.2.pyc
│ ├── integration_test.cpython-314-pytest-9.0.2.pyc
│ ├── test_fourigui.cpython-314-pytest-9.0.2.pyc
│ └── test_version.cpython-314-pytest-9.0.2.pyc
├── conftest.py
├── integration_test.py
├── test_fourigui.py
├── test_version.py
└── testdata
├── __init__.py
├── dummydata.h5
├── gofr.h5
├── gofr_.h5
├── gofr_cut.h5
├── gofr_from_sofq_cut_10to40px.h5
├── gofr_from_sofq_cut_15to35px.h5
├── make_testdata.py
├── sofq.h5
├── sofq_cut_10to40px.h5
└── sofq_cut_15to35px.h5
36 directories, 156 files

@codecov

codecovBot commented Apr 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.78%. Comparing base (609db53) to head (f0e31c7).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #94 +/- ##
=======================================
Coverage 93.78% 93.78% =======================================
Files 4 4 Lines 177 177 =======================================
Hits 166 166 Misses 11 11 
Files with missing linesCoverage Δ
tests/integration_test.py98.03% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sbillinge

Copy link
Copy Markdown
Contributor

I think maybe just the testdata filenames need to be changed to having dashes?

@sbillinge
sbillinge merged commit 2ab32d1 into diffpy:mainApr 7, 2026
7 checks passed
@sbillinge

Copy link
Copy Markdown
Contributor

yipee, thanks!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants

@stevenhua0320@sbillinge