Skip to content

Update readme#2

Open
LigiaZ wants to merge 5 commits intomainfrom
update-readme
Open

Update readme#2
LigiaZ wants to merge 5 commits intomainfrom
update-readme

Conversation

@LigiaZ
Copy link
Owner

@LigiaZ LigiaZ commented Nov 26, 2025

No description provided.

- Add comprehensive conversion details for Temperature, Length, Volume, and Weight categories
- Include mathematical formulas and conversion factors
- Document all available conversion pairs (Celsius ↔ Fahrenheit/Kelvin, Kilometers ↔ Miles/Yards, etc.)
- Provide user interface information and setup instructions
- Improve documentation clarity with structured format
Copy link

@kiloconnect kiloconnect bot left a comment

Choose a reason for hiding this comment

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

⚠️ 2 Issues Found

Severity Issue Location
WARNING Invalid requirements.txt syntax requirements.txt:1
SUGGESTION Typo in description README.md:2

Recommendation: Address the warning issue before merge

Review Details (2 files)

Files: README.md (1 issue), requirements.txt (1 issue)

@@ -1 +1 @@
streamlit No newline at end of file
streamlit necessary to have No newline at end of file
Copy link

Choose a reason for hiding this comment

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

WARNING: Invalid requirements.txt syntax

The line "streamlit necessary to have " is not valid pip syntax. Requirements files should contain package names only, optionally with version specifiers. This will cause pip install to fail. Revert to "streamlit" or specify a version like "streamlit==1.28.1".

@@ -1,15 +1,47 @@
# UnitConverter
Unit converter project with my husband
Copy link

Choose a reason for hiding this comment

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

SUGGESTION: Typo in description

"The sweeetest!" appears to be a typo for "The sweetest!". Please correct the spelling.

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