Skip to content
Snippets Groups Projects

Add Go guidelines to developer docs

Merged Philippe Lafoucrière requested to merge docs/go-standards into master

What does this MR do?

Adds our Go guidelines to the development doc.

Related issues

Closes #54494 (closed)

Author's checklist

Review checklist

  • Your team's review (required)
  • PM's review (recommended, but not a blocker) (development guidelines)
  • Technical writer's review (required)
  • Merge the EE-MR first, CE-MR afterwards

/label ~Documentation golang development guidelines

Edited by Marcia Ramos

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Steve Xuereb
  • Steve Xuereb
  • Steve Xuereb
  • Steve Xuereb
  • added 1 commit

    Compare with previous version

  • added 1 commit

    • 48901dcb - Update tracing and Correlation title

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Philippe Lafoucrière changed the description

    changed the description

  • @plafoucriere when do you think we can get this MR merged? I want to include it in a legal doc but don't want to wait too long.

  • @eliran.mesika I will polish some details today (the left TODOs), and we should be able to merge tomorrow. Does that work for you? @marin could you please take a look? We can always update and add content later, but at least @eliran.mesika will have a page to link in his docs. Thanks

  • @plafoucriere tomorrow is good, thank you. Having a page we can reference to and note it'll undergo updates in the future is key.

  • added 2 commits

    • 40cf5322 - Add note on go >= 1.11.5 requirement
    • a510426e - Fix formatting and wording

    Compare with previous version

  • Philippe Lafoucrière unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Marin Jankovski
  • Good starting point @plafoucriere, thanks!

  • changed milestone to %11.8

  • Fabien Catteau
  • Fabien Catteau
  • Fabien Catteau
  • mentioned in issue #57020 (moved)

  • added 1 commit

    Compare with previous version

  • @marcia could you please do a final review? Thanks

  • mentioned in issue #57045 (moved)

  • Marcia Ramos
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading