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

Update semantic-conventions Directory to Support PyPi Packaging #255

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update semantic-conventions Directory to Support PyPi Packaging #255

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions semantic-conventions/README.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
OpenTelemetry Semantic Convention Utility Tools
===============================================

|pypi|

.. |pypi| image:: https://badge.fury.io/py/opentelemetry-semconvgen.svg
:target: https://pypi.org/project/opentelemetry-semconvgen/

This library is provided as a convenience to access the OpenTelemetry Semantic Convention Generator, and semantic convention syntax parsing.

Installation
------------

::

pip install opentelemetry-semconvgen
12 changes: 6 additions & 6 deletions semantic-conventions/setup.cfg
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,15 @@
# limitations under the License.
#
[metadata]
name = semconvgen
description = OpenTelemetry Semantic Conventions utility
name = opentelemetry-semconvgen
description = OpenTelemetry Semantic Conventions Utilities
author = The OpenTelemetry Authors
author_email = cncf-opentelemetry-contributors@lists.cncf.io
url = https://github.com/dynatrace-oss-contrib/build-tools/
url = https://github.com/open-telemetry/build-tools
platforms = any
license = Apache-2.0
classifiers =
Development Status :: 1 - Alpha
Development Status :: 3 - Alpha
Intended Audience :: Developers
License :: OSI Approved :: Apache Software License
Programming Language :: Python
Expand All @@ -36,10 +36,10 @@ package_dir=
=src
packages=find_namespace:
install_requires =
dataclasses~=0.6
dataclasses~=0.6;python_version<'3.7'
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we support python < 3.7 at all, probably we need the Python version we currently run with in the container as minimum.

Copy link
Author

Choose a reason for hiding this comment

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

hmm, got it. I added that part just because dataclasses comes standard in 3.7, so we wouldn't have to install it in that case.

ruamel.yaml~=0.16
Jinja2~=3.0
mistune==2.0.0a6
mistune==2.0.0

[options.packages.find]
where = src
Expand Down
10 changes: 9 additions & 1 deletion semantic-conventions/setup.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,16 @@
VERSION_SUFFIX = os.environ.get("SEMCONGEN_VERSION_SUFFIX")
PUBLIC_VERSION = PACKAGE_INFO["__version__"]

# long description
README_FILENAME = os.path.join(BASE_DIR, "README.rst")
LONG_DESCRIPTION = ""
with open(README_FILENAME, encoding="utf-8") as f:
LONG_DESCRIPTION = f.read()

setuptools.setup(
version=PUBLIC_VERSION
if not VERSION_SUFFIX
else PUBLIC_VERSION + "+" + VERSION_SUFFIX
else PUBLIC_VERSION + "+" + VERSION_SUFFIX,
long_description=LONG_DESCRIPTION,
long_description_content_type="text/x-rst",
)