Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add clarity to the "Build from source" option#264

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
toby merged 1 commit intomainfromrfearing/update-readme-local
Apr 14, 2025

Conversation

rfearing
Copy link
Contributor

In working on contributing to#225, I found this area of the instructions could use more clarity. Hopefully this is clearer and provides additional instructions on how to specify the output location of the build and configure the server to use the built executable.

@rfearingrfearing changed the titleClarify local buildAdd clarity to the "Build from source" optionApr 14, 2025
@rfearingrfearing marked this pull request as ready for reviewApril 14, 2025 15:03
@CopilotCopilotAI review requested due to automatic review settingsApril 14, 2025 15:03
Copy link
Contributor

@CopilotCopilotAI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR clarifies the "Build from source" instructions in the README by explicitly describing how to build the executable and configure the server to use it.

  • Updated instructions to use "go build" instead of "go".
  • Added guidance on specifying an output location via the "-o" flag along with an example JSON configuration.

@StefMa
Copy link

Same/similar as/to#224

Copy link
Member

@tobytoby left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@tobytoby merged commit62eed34 intomainApr 14, 2025
16 checks passed
@tobytoby deleted the rfearing/update-readme-local branchApril 14, 2025 18:18
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

Copilot code reviewCopilotCopilot left review comments

@tobytobytoby approved these changes

@juruenjuruenAwaiting requested review from juruen

@SamMorrowDrumsSamMorrowDrumsAwaiting requested review from SamMorrowDrums

@williammartinwilliammartinAwaiting requested review from williammartin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@rfearing@StefMa@toby

[8]ページ先頭

©2009-2025 Movatter.jp