Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(github-actions)🔧: Update deployment workflow to use setup-pixi action #443

Merged
merged 19 commits into from
Mar 4, 2025

Conversation

ericmjl
Copy link
Owner

@ericmjl ericmjl commented Feb 23, 2025

  • Replaced the setup-conda action with setup-pixi for environment setup.
  • Configured pixi-specific parameters including version, cache, and authentication.

- Replaced the setup-conda action with setup-pixi for environment setup.
- Configured pixi-specific parameters including version, cache, and authentication.
…ps from deploy workflow

- Deleted steps related to caching and building the conda environment.
- Updated the deploy workflow to streamline the process by removing unnecessary steps.
- Replaced the direct bash script execution with pixi for building the website.
- Removed redundant steps related to environment setup in the deployment workflow.
…build script

- Commented out the Leanpub deployment steps in the GitHub Actions workflow.
- Deleted the script for building the MkDocs website as it is no longer required.
…rkflows

- Removed the 'check.yaml' workflow as its functionality is now integrated into 'deploy.yaml'.
- Updated 'deploy.yaml' to include pull request triggers and conditional deployment steps.
- Removed unused authentication parameters from the deploy workflow.
- Added a new linting workflow for code style checks.
- Updated the GitHub Actions workflow to trigger only on push and pull requests targeting the master branch.
- Replaced the 'pixi run bash scripts/ci/build_website.sh' command with 'pixi run build-docs' in the deploy workflow.
- This change simplifies the build process by using a more direct command.
- Included a command to install a Python kernel in the deployment workflow.
- This ensures the required environment is set up before building the documentation.
- Changed the kernel display name from 'default' to 'nams'.
- Updated the kernel name from 'python3' to 'nams'.
…iguration

- Removed the older nbstripout hook version 0.6.1.
- Added the updated nbstripout hook version 0.8.1.
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

ericmjl added 6 commits March 3, 2025 20:27
- Updated the pre-commit-hooks repository to version 5.0.0.
- Updated the black repository to version 25.1.0.
- Inserted a blank line at the beginning of the file to separate the docstring from the imports.
…rsion in notebook

- Cleaned up unnecessary blank lines in the code cells for better readability.
- Updated the Python version metadata to reflect the current environment.
…algorithm structures.

- Fixed the sampling method for nodes in the test_in_triangle function.
- Removed unnecessary trailing whitespace in code cells.
- Updated the Python version metadata in the notebook.
…stency

- Updated string delimiters to use double quotes consistently.
- Reformatted multi-line statements for improved readability.
- Ensured consistent spacing and indentation across all cells.
… workflow

- Introduced a new step in the deploy job to handle PR preview deployment.
- Utilized the rossjrw/pr-preview-action for generating previews for pull requests.
Copy link

github-actions bot commented Mar 4, 2025

PR Preview Action v1.6.0

🚀 View preview at
https://ericmjl.github.io/Network-Analysis-Made-Simple/pr-preview/pr-443/

Built to branch gh-pages at 2025-03-04 02:04 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@ericmjl ericmjl merged commit f187a5e into master Mar 4, 2025
3 checks passed
@ericmjl ericmjl deleted the upgrade-cicd-pixi branch March 4, 2025 02:06
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.

1 participant