[wishlist] Update grape rubygems to 1.7
We'd like to use grape 1.6.2 in debian, though grape 1.6.2 no longer has Grape::Validations::Base and rake db:migrate fails due to this.
As a workaround we can use grape 1.5.2 for now, but it'd be good if we can use grape 1.6.2.
Update: 1.7.0 may be a better option as it fixed the regression in 1.6.x versions
Designs
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Author Contributor
though it looks like a regression and this was added back in grape 1.7.0
- Author Contributor
Though in 1.7.0 they marked it as deprecated. https://github.com/ruby-grape/grape/commit/9687a88bafe8e75547f1fe5cb19e626b429ce475#diff-b7f444ec36760579c6636601e285c717058f27efcfc93a232f1f69897b7eff7bR100
warn '[DEPRECATION] `Grape::Validations::Base` is deprecated. Use `Grape::Validations::Validators::Base` instead.'
- Contributor
This issue was automatically tagged with the label groupdatabase by TanukiStan, a machine learning classification model, with a probability of 0.46.
If this label is incorrect, please tag this issue with the correct group label as well as automation:ml wrong to help TanukiStan learn from its mistakes.
If you are unsure about the correct group, please do not leave the issue without a group label. Please refer to GitLab's shared responsibility functionality guidelines for more information on how to triage this kind of issues.
Authors who do not have permission to update labels can leave the issue to be triaged by group leaders initially assigned by TanukiStan
This message was generated automatically. You're welcome to improve it.
- A deleted user added automation:ml groupdatabase labels
added automation:ml groupdatabase labels
- 🤖 GitLab Bot 🤖 added devopsdata stores sectioncore platform labels
added devopsdata stores sectioncore platform labels
- Maintainer
Hey @pravi, thank you for creating this issue!
To get the right eyes on your issue more quickly, we encourage you to follow the issue triage best practices.
PLEASE NOTE: You will need to use the
@gitlab-bot label command
to apply labels to this issue.To set expectations, GitLab product managers or team members can't make any promise if they will proceed with this. However, we believe everyone can contribute, and welcome you to work on this proposed change, feature or bug fix. There is a bias for action, so you don't need to wait. Try and spin up that merge request yourself.
If you need help doing so, we're always open to mentor you to drive this change.
This message was generated automatically. You're welcome to improve it.
- 🤖 GitLab Bot 🤖 added automation:self-triage-encouraged label
added automation:self-triage-encouraged label
- Maintainer
Setting label(s) Category:Database based on groupdatabase.
- 🤖 GitLab Bot 🤖 added Category:Database label
added Category:Database label
- Alex Ives mentioned in issue gitlab-org/database-team/team-tasks#336 (closed)
mentioned in issue gitlab-org/database-team/team-tasks#336 (closed)
- Leonardo da Rosa added groupauthentication and authorization [DEPRECATED] label and removed groupdatabase label
added groupauthentication and authorization [DEPRECATED] label and removed groupdatabase label
- 🤖 GitLab Bot 🤖 added devopsmanage sectiondev labels and removed devopsdata stores sectioncore platform labels
added devopsmanage sectiondev labels and removed devopsdata stores sectioncore platform labels
- Leonardo da Rosa removed Category:Database label
removed Category:Database label
- Maintainer
/cc @hsutor @adil.farrukh could you take a look if Authentication and Authorization team could take action on this one?
Collapse replies - Developer
@l.rosa Thanks for flagging this. Grape was recently review by ~"group::integrations" based on https://docs.google.com/spreadsheets/d/1gweXRsv0MdMXMyejuWpmGmp2T123geq2rlsJrdDh8r0/edit#gid=0 but it's used across the board (& can impact other teams) so will wait for their input & relabel if needed
1
- Praveen Arimbrathodiyil changed title from [wishlist] Update grape rubygems to 1.6.2 to [wishlist] Update grape rubygems to 1.7
changed title from [wishlist] Update grape rubygems to 1.6.2 to [wishlist] Update grape rubygems to 1.7
- Praveen Arimbrathodiyil changed the description
Compare with previous version changed the description
removed groupauthentication and authorization [DEPRECATED] label
- Adil Farrukh added groupintegrations [DEPRECATED] label
added groupintegrations [DEPRECATED] label
- 🤖 GitLab Bot 🤖 added groupimport and integrate label and removed groupintegrations [DEPRECATED] label
added groupimport and integrate label and removed groupintegrations [DEPRECATED] label
- Maintainer
I was working on Grape update to version 1.7 and I have a draft MR for it: !112854 (merged). There are a few changes that we need to apply to our API but it's close to have a green test suite.
I will assign this issue to myself and try to complete the version update.
- Vasilii Iakliushin assigned to @vyaklushin
assigned to @vyaklushin
- Vasilii Iakliushin mentioned in commit 4c950c98
mentioned in commit 4c950c98
- Vasilii Iakliushin mentioned in merge request !121694 (merged)
mentioned in merge request !121694 (merged)
- Vasilii Iakliushin mentioned in commit 3cf8758b
mentioned in commit 3cf8758b
- Vasilii Iakliushin mentioned in merge request !121700 (merged)
mentioned in merge request !121700 (merged)
- Vasilii Iakliushin mentioned in commit c8d7a4e4
mentioned in commit c8d7a4e4
- Vasilii Iakliushin added backend groupsource code workflowin review labels and removed groupimport and integrate label
added backend groupsource code workflowin review labels and removed groupimport and integrate label
- 🤖 GitLab Bot 🤖 added devopscreate label and removed devopsmanage label
added devopscreate label and removed devopsmanage label
- Vasilii Iakliushin changed milestone to %16.1
changed milestone to %16.1
- Vasilii Iakliushin mentioned in merge request !121705 (merged)
mentioned in merge request !121705 (merged)
- Maintainer
@vyaklushin - please see the following guidance and update this issue.1 Error Please add typebug typefeature, typemaintenance and a subtype label to this issue. If you do not feel the purpose of this issue matches one of the types, you may apply the typeignore label to exclude it from type tracking metrics and future prompts.
This message was generated automatically. You're welcome to improve it.
- Vasilii Iakliushin added maintenancedependency label
added maintenancedependency label
- 🤖 GitLab Bot 🤖 added typemaintenance label
added typemaintenance label
- Vasilii Iakliushin mentioned in commit cb66be7a
mentioned in commit cb66be7a
- Vasilii Iakliushin mentioned in commit f1f91640
mentioned in commit f1f91640
- Maintainer
Setting label(s) Category:Source Code Management based on groupsource code.
- 🤖 GitLab Bot 🤖 added Category:Source Code Management label
added Category:Source Code Management label
- Pam Artiaga mentioned in merge request !121791 (merged)
mentioned in merge request !121791 (merged)
- Vasilii Iakliushin mentioned in commit d809c19e
mentioned in commit d809c19e
- Vasilii Iakliushin mentioned in commit cd07be55
mentioned in commit cd07be55
- Vasilii Iakliushin mentioned in commit 47f27476
mentioned in commit 47f27476
- Vasilii Iakliushin mentioned in merge request !122061 (merged)
mentioned in merge request !122061 (merged)
- Vasilii Iakliushin mentioned in commit a28b2b62
mentioned in commit a28b2b62
- Vasilii Iakliushin mentioned in commit 21f63f21
mentioned in commit 21f63f21
- Vasilii Iakliushin mentioned in commit 6f39ef0d
mentioned in commit 6f39ef0d
- Vasilii Iakliushin mentioned in merge request !112854 (merged)
mentioned in merge request !112854 (merged)
- Amy Qualls added twtriaged label
added twtriaged label
- Vasilii Iakliushin added workflowverification label and removed workflowin review label
added workflowverification label and removed workflowin review label
- Maintainer
Grape version 1.7.0 is deployed to GitLab.com and will be released in %16.1.
Marking this issue as complete.
- Vasilii Iakliushin added workflowcomplete label and removed workflowverification label
added workflowcomplete label and removed workflowverification label
- Vasilii Iakliushin closed
closed
- Vasilii Iakliushin mentioned in commit bdd30c47
mentioned in commit bdd30c47
- Vasilii Iakliushin mentioned in merge request !123779 (merged)
mentioned in merge request !123779 (merged)
- Vasilii Iakliushin mentioned in commit 00f5fcc4
mentioned in commit 00f5fcc4
- Vasilii Iakliushin mentioned in commit 954485c0
mentioned in commit 954485c0