Skip to content

skpkg: update README.rst installation instructions and other minor fixes#41

Merged
sbillinge merged 4 commits into
diffpy:mainfrom
zmx27:update-workflows
Nov 8, 2025
Merged

skpkg: update README.rst installation instructions and other minor fixes#41
sbillinge merged 4 commits into
diffpy:mainfrom
zmx27:update-workflows

Conversation

@zmx27

@zmx27 zmx27 commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@zmx27

zmx27 commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author

@sbillinge ready for review

Comment thread .github/workflows/build-wheel-release-upload.yml
@@ -1,12 +0,0 @@
name: Deploy Documentation on Release

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is now unnecessary. This functionality is bundled with the build-wheel workflow file

Comment thread requirements/conda.txt
fabio
tifffile
configparser
diffpy.srxconfutils

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should I delete this for now so CI can pass? Since srxconfutils is not on CF

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, let's leave it here for now. We can do the release for this when that is available on cf

@zmx27 zmx27 mentioned this pull request Nov 7, 2025
18 tasks

@sbillinge sbillinge left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will merge this.

Comment thread requirements/conda.txt
fabio
tifffile
configparser
diffpy.srxconfutils

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, let's leave it here for now. We can do the release for this when that is available on cf

@sbillinge sbillinge merged commit 584597e into diffpy:main Nov 8, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants