Skip to content
This repository has been archived by the owner on Jan 29, 2025. It is now read-only.

✅ Build Script: Write a script that build & deploy the package #188

Closed
wants to merge 5 commits into from
Closed

✅ Build Script: Write a script that build & deploy the package #188

wants to merge 5 commits into from

Conversation

yashksaini-coder
Copy link
Contributor

This PR after further testing, has finally fixed the issue #167 to automate the Build & Deploy script.

IT uses the previous approach we discussed in #167,

sneaky-package-167.webm

@this-is-yaash Review this PR and the code changes and kindly assign labels to this PR.

This pull request includes significant changes to the build process and documentation of the project. The most important changes are the addition of a new build script, removal of contribution guidelines, and updates to the setup.py file.

Build Process Improvements:

  • build.py: Introduced a new build script to dynamically create setup.py, build the package, and upload it to PyPI using environment variables.

Documentation Changes:

  • docs/Contributing.md: Removed the detailed contribution guidelines, which included instructions for setting up the development environment, understanding the codebase, and submitting contributions.
  • docs/contributing.md: Removed the detailed contribution guidelines, similar to the changes in docs/Contributing.md.

Package Configuration:

  • setup.py: Updated package metadata, including the package name, version, description, author name, and author email.

.gitignore Outdated Show resolved Hide resolved
@iamyaash
Copy link
Collaborator

@yashksaini-coder I think we don't have to enter the token when prompting.
Make it fetch from the .env.

Then we'll merge it!

Copy link
Collaborator

Choose a reason for hiding this comment

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

don't delete this file!
!1

Copy link
Collaborator

Choose a reason for hiding this comment

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

don't delete this file!
!2

Copy link
Collaborator

Choose a reason for hiding this comment

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

revert back to original changes

@yashksaini-coder
Copy link
Contributor Author

We can discuss the progress and work done so far, and resolving the Contributing.md file issue tomorrow, for now closing this PR

@yashksaini-coder yashksaini-coder deleted the test-buil_deploy branch November 1, 2024 04:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants