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.

Sign up for GitHub

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

Jump to bottom

Move tjson into internal/handlers/tigris/tjson #1252

Closed
AlekSi opened this issue Oct 12, 2022 · 1 comment · Fixed by #1291
Closed

Move tjson into internal/handlers/tigris/tjson #1252

AlekSi opened this issue Oct 12, 2022 · 1 comment · Fixed by #1291
Assignees
@chilagrow
Labels
code/chore Code maintenance improvements
Milestone
v0.6.0 Alpha

Comments

@AlekSi
Copy link
Member

AlekSi commented Oct 12, 2022

What should be done?

To make tigris handler looks more like pg handler, let's move tjson package into internal/handlers/tigris/tjson.

We should update:

  • code location;
  • code comments;
  • documentation comments;
  • CONTRIBUTING.md;
  • golangci-lint's depguard rules;
  • etc.

Where?

Search for "tjson" and check everything.
@AlekSi AlekSi added code/chore Code maintenance improvements code/tigris labels Oct 12, 2022
@rumyantseva
Copy link
Member

Blocked by #943, to be taken after #943 is done.

@chilagrow chilagrow self-assigned this Oct 20, 2022
@chilagrow chilagrow mentioned this issue Oct 20, 2022
Merged
7 tasks
chilagrow added a commit that referenced this issue Oct 21, 2022
@chilagrow
Move tjson into internal/handlers/tigris/tjson
2555b09 
Closes #1252.
@AlekSi AlekSi added this to the v0.6.0 Alpha milestone Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chilagrow chilagrow

Labels
code/chore Code maintenance improvements
Projects
None yet
Milestone
v0.6.0 Alpha
Development

Successfully merging a pull request may close this issue.

Move tjson into internal/handlers/tigris/tjson chilagrow/FerretDB
3 participants
@AlekSi @rumyantseva @chilagrow