Refactor base.py and Dockerfile, and folder column migration #2002
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…unction from langflow.version module to prevent crashing the application feat(base.py): dynamically determine if the version is a pre-release version by checking for 'a', 'b', or 'rc' in the version number
…mically determine the version number The get_lifespan function in main.py has been updated to handle ImportError when importing the __version__ attribute from the langflow.version module. If the import fails, the version number is dynamically determined using the importlib.metadata.version function. This change ensures that the application does not crash when the langflow.version module is not available. Note: This commit message follows the convention used in the recent user commits.
… migration This commit adds the folder_id column to the flow table in the database. It also handles the migration of the folder column to the new folder_id column. This change allows for better organization and management of flows within folders. Note: This commit message follows the convention used in the recent user commits.
… variable This commit updates the Dockerfiles to include the creation of a user with UID 1000 and the necessary permissions for the /app/langflow directory. It also adds the user's local bin directory to the PATH environment variable. This change improves the security and isolation of the application within the container. Note: This commit message follows the convention used in the recent user commits.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
dosubot
bot
added
size:M
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request includes several changes to improve the codebase and functionality of the application.
The first commit refactors the base.py file to handle ImportError when importing the is_pre_release function from the langflow.version module. This prevents the application from crashing when the module is not available. Additionally, the commit adds dynamic determination of whether the version is a pre-release version by checking for 'a', 'b', or 'rc' in the version number.
The second commit fixes a missing backslash in the Dockerfile, ensuring proper chaining of commands.
The third commit updates the Dockerfiles to include the creation of a user with UID 1000 and the necessary permissions for the /app/langflow directory. It also adds the user's local bin directory to the PATH environment variable, improving security and isolation within the container.
The fourth commit adds the folder_id column to the flow table in the database and handles the migration of the folder column to the new folder_id column. This change allows for better organization and management of flows within folders.
Please review and merge these changes as appropriate.