Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make staticrequired shouldn't require all kinds of local installs, fixes #2253 #2254

Merged
merged 3 commits into from May 15, 2020

Conversation

rfay
Copy link
Member

@rfay rfay commented May 14, 2020

The Problem/Issue/Bug:

#2253 points out that make staticrequired requires lots of workstation-installed software, and it shouldn't be like that.

How this PR Solves The Problem:

  • Update golang via build-tools and golang-build-container
  • mkdocs and markdownlint-cli are now in the golang-build-container
  • Change the githooks and make staticrequired to just use make staticrequired

Manual Testing Instructions:

make staticrequired - no local installs required beyond go and docker.

Automated Testing Overview:

Related Issue Link(s):

Release/Deployment notes:

@rfay rfay changed the title 20200513 bump build tools golang, fixes #2253 make staticrequired shouldn't require all kinds of local installs, fixes #2253 May 14, 2020
@rfay rfay linked an issue May 14, 2020 that may be closed by this pull request
@rfay rfay merged commit 3e99c24 into ddev:master May 15, 2020
@rfay rfay deleted the 20200513_bump_build_tools_golang branch May 15, 2020 23:29
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.

Document or simplify the tooling for staticrequired
1 participant