Skip to content

[api][ci] Introduce paimon_python_api and ci framework#2

Merged
JingsongLi merged 2 commits intoapache:mainfrom
yuzelin:add-ci
Aug 15, 2024
Merged

[api][ci] Introduce paimon_python_api and ci framework#2
JingsongLi merged 2 commits intoapache:mainfrom
yuzelin:add-ci

Conversation

@yuzelin
Copy link
Copy Markdown
Contributor

@yuzelin yuzelin commented Aug 14, 2024

  1. Define python API
  2. Introduce ci framework, containing code style check and test.

TODO:

  1. docs & docs check
  2. building workflow

Comment thread setup.py Outdated
install_requires=[],
description='Apache Paimon Python API',
author='Apache Software Foundation',
author_email='dev@flink.apache.org',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

paimon

Comment thread dev/lint-python.sh Outdated
install_environment
fi

pushd "$FLINK_PYTHON_DIR" &> /dev/null
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PAIMON

Comment thread dev/lint-python.sh Outdated
# CURRENT_DIR is "paimon-python/dev/"
CURRENT_DIR="$(cd "$( dirname "$0" )" && pwd)"

# FLINK_PYTHON_DIR is "flink/flink-python"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is not flink dir

Comment thread .github/PULL_REQUEST_TEMPLATE.md Outdated
@@ -0,0 +1,20 @@
<!-- Please specify the module before the PR name: [core] ... or [flink] ... -->
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove this line?

@JingsongLi
Copy link
Copy Markdown
Contributor

+1

@JingsongLi JingsongLi merged commit 8e07985 into apache:main Aug 15, 2024
@yuzelin yuzelin deleted the add-ci branch October 10, 2024 04:14
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