Update Linux build instructions #1454
Conversation
WalkthroughThe recent update to the Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
![coderabbitai[bot]](/~https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Outside diff range and nitpick comments (5)
README.md (5)
Line range hint
14-14: Ensure consistency in word usage: choose either "utilises" or "utilize" throughout the document to maintain style coherence.
Line range hint
16-16: Consider adding a comma after "system" and "algorithm" for better readability.
Line range hint
20-20: Remove the repeated word "the" to correct the sentence: "then running Firo with Docker might be the easier method for you."
Line range hint
173-173: Consider rephrasing for better flow: "Note: These instructions are inherited from Bitcoin and may not work as expected."
Line range hint
196-221: Add alternate text to the images to improve accessibility.
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- README.md (1 hunks)
Additional context used
LanguageTool
README.md
[uncategorized] ~14-~14: Do not mix variants of the same word (‘utilise’ and ‘utilize’) within a single text. (EN_WORD_COHERENCY)
Context: ...pers/LinfengSparkAudit.pdf). Firo also utilises [Dandelion++](https://arxiv.org/abs/180...
[uncategorized] ~16-~16: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...o uses a hybrid PoW and LLMQ Chainlocks system combining fair distribution of supply w...
[uncategorized] ~16-~16: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...W variant) is used as its Proof-of-Work algorithm which targets GPUs and is FPGA/ASIC res...
[duplication] ~20-~20: Possible typo: you repeated a word (ENGLISH_WORD_REPEAT_RULE)
Context: ... then running Firo with Docker might be the the easier method for you. To run Firo usin...
[typographical] ~20-~20: Consider adding a comma here. (TO_THAT_COMMA)
Context: ...type=edition&offering=community). After this you may continue with the following ins...
[grammar] ~98-~98: The verb ‘depend’ requires the preposition ‘on’ (or ‘upon’). (DEPEND_ON)
Context: ...ns such as cross compilation, read the depends instructions Alter...
[style] ~173-~173: To make your writing flow more naturally, try moving ‘also’ before the verb. (ALSO_PLACEMENT)
Context: ...f you chose to build the GUI,firo-qtwill be also generated in theqtfolder. ## macOS...
[grammar] ~189-~189: The verb ‘to depend’ requires the preposition ‘(up)on’. (DEPEND_ON)
Context: ...mpiled binaries, it's time to run Firo! Depending by your skill level and/or setup, you migh...
Markdownlint
README.md
18-18: null (MD025, single-title, single-h1)
Multiple top-level headings in the same document
95-95: null (MD025, single-title, single-h1)
Multiple top-level headings in the same document
187-187: null (MD025, single-title, single-h1)
Multiple top-level headings in the same document
191-191: null (MD025, single-title, single-h1)
Multiple top-level headings in the same document
125-125: null (MD031, blanks-around-fences)
Fenced code blocks should be surrounded by blank lines
126-126: null (MD032, blanks-around-lists)
Lists should be surrounded by blank lines
179-179: null (MD036, no-emphasis-as-heading)
Emphasis used instead of a heading
185-185: null (MD036, no-emphasis-as-heading)
Emphasis used instead of a heading
196-196: null (MD045, no-alt-text)
Images should have alternate text (alt text)
206-206: null (MD045, no-alt-text)
Images should have alternate text (alt text)
212-212: null (MD045, no-alt-text)
Images should have alternate text (alt text)
213-213: null (MD045, no-alt-text)
Images should have alternate text (alt text)
214-214: null (MD045, no-alt-text)
Images should have alternate text (alt text)
215-215: null (MD045, no-alt-text)
Images should have alternate text (alt text)
216-216: null (MD045, no-alt-text)
Images should have alternate text (alt text)
217-217: null (MD045, no-alt-text)
Images should have alternate text (alt text)
218-218: null (MD045, no-alt-text)
Images should have alternate text (alt text)
219-219: null (MD045, no-alt-text)
Images should have alternate text (alt text)
220-220: null (MD045, no-alt-text)
Images should have alternate text (alt text)
221-221: null (MD045, no-alt-text)
Images should have alternate text (alt text)
Additional comments not posted (1)
README.md (1)
115-115: The update to specifypython3is clear and relevant for Ubuntu users. Consider adding a specific Ubuntu version to clarify from which version this change is necessary.
Recent versions of Ubuntu require
python3be specified as a dependency instead ofpython. This PR updates the documentation accordingly.Closes #1452.