Skip to content

api: add hardcoded versioning support #62

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

Merged

Conversation

GRISHNOV
Copy link
Contributor

@GRISHNOV GRISHNOV commented Mar 15, 2023

Add versioning support, now the module api has _VERSION hardcoded variable. Is part of the task [1].

  1. https://github.com/tarantool/roadmap-internal/issues/204

@GRISHNOV
Copy link
Contributor Author

GRISHNOV commented Mar 15, 2023

note: Run tests / all (push) in ci/cd have a problem that not caused by the current task

@GRISHNOV GRISHNOV force-pushed the igrishnov/gh-204-add-hardcoded-versioning-support branch from c38e945 to ef31f34 Compare March 15, 2023 09:29
Added the _VERSION variable to the exported table.
Is part of the task [1].

1. github.com/tarantool/roadmap-internal/issues/204
@GRISHNOV GRISHNOV force-pushed the igrishnov/gh-204-add-hardcoded-versioning-support branch from ef31f34 to eaf7c3f Compare March 15, 2023 09:34
@DifferentialOrange
Copy link
Member

note: Run tests / all (push) in ci/cd have a problem that not caused by the current task

It seems like it could be fixed with one-line commit. Let's do it

@DifferentialOrange
Copy link
Member

Bump test workflow OS to Ubuntu 20.04 and install gcc/g++. If there are more work will be discovered, let's ignore it.

@GRISHNOV GRISHNOV force-pushed the igrishnov/gh-204-add-hardcoded-versioning-support branch from d69fc6a to 766bd09 Compare March 15, 2023 13:55
Copy link
Member

@DifferentialOrange DifferentialOrange left a comment

Choose a reason for hiding this comment

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

Let's not fix CI any longer. File an issue, tarantool/http@c08227d is likely to be the reason. You may also add test xfail/skip to get a green light.

@DifferentialOrange DifferentialOrange marked this pull request as ready for review March 15, 2023 14:47
@GRISHNOV GRISHNOV force-pushed the igrishnov/gh-204-add-hardcoded-versioning-support branch from 766bd09 to eaf7c3f Compare March 15, 2023 16:33
@GRISHNOV
Copy link
Contributor Author

Let's not fix CI any longer. File an issue, tarantool/http@c08227d is likely to be the reason. You may also add test xfail/skip to get a green light.

Done

GRISHNOV and others added 2 commits March 16, 2023 15:43
Git222 is deprecated.
Also, to build dependent modules,
an ubuntu bump was needed and added
gcc/gcc-c++ to centos packages installation.
@DifferentialOrange DifferentialOrange force-pushed the igrishnov/gh-204-add-hardcoded-versioning-support branch from 37b7adb to b275349 Compare March 16, 2023 14:31
@DifferentialOrange DifferentialOrange merged commit bf74eec into master Mar 17, 2023
@DifferentialOrange DifferentialOrange deleted the igrishnov/gh-204-add-hardcoded-versioning-support branch March 17, 2023 08:20
DifferentialOrange added a commit that referenced this pull request Mar 20, 2023
Overview

  This release introduces new API to check module version in code.

New features
  * Add versioning support (PR #62).
@DifferentialOrange DifferentialOrange mentioned this pull request Mar 20, 2023
DifferentialOrange added a commit that referenced this pull request Mar 22, 2023
Overview

  This release introduces new API to check module version in code.

New features
  * Add versioning support (PR #62).
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.

2 participants