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

Review Go guide #891

Merged
merged 1 commit into from
May 14, 2024
Merged

Review Go guide #891

merged 1 commit into from
May 14, 2024

Conversation

paroxp
Copy link
Member

@paroxp paroxp commented Apr 23, 2024

The contents are still valid in my belief.

I would like us to consider and look at deprecating this section as a whole possibly next year. I could have suggested it now, however I believe there is still a lot of potential for Go in GDS - especially in the GOV.UK's K8s setup. Not to mention the GDS CLI and remains of PaaS.

I imagine GOV.UK may want to have some custom operators being written and maintained in Go, which may mean deprecating this guide isn't necessarily the right call. But perhaps, the direction the team is going at will constraint the use of Go to simply running an Open Source projects.

Copy link
Member

@huwd huwd left a comment

Choose a reason for hiding this comment

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

Probably worth getting a review of someone on the GOV.UK side as there's no Go in my patch.

But I think the logic is sound @paroxp, approving on that basis

@timblair
Copy link
Member

timblair commented May 9, 2024

We've already removed Go from the list of core backend languages, in #876.

@paroxp
Copy link
Member Author

paroxp commented May 9, 2024

Yeah, I've seen that :) Now to deprecate this section :D

@paroxp paroxp merged commit 35a5aa9 into main May 14, 2024
1 check passed
@paroxp paroxp deleted the golang-bump branch May 14, 2024 15:12
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.

3 participants