Skip to content

Conversation

@Freeza91
Copy link

No description provided.

Copy link
Contributor

@superisaac superisaac left a comment

Choose a reason for hiding this comment

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

相关 requirement 需要写到setup.py 里,还需要更新git tag

bin/bbox.py Outdated
help_msg = getattr(handler, 'help', '')
parser = sub_parsers.add_parser(sub_cmd, help=help_msg)
# add sentry config
parser.add_argument(
Copy link
Contributor

Choose a reason for hiding this comment

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

通过环境变量 SENTRY_URL 来设置就可以了

bin/bbox.py Outdated
args = top_parser.parse_args(input_args)
loop = asyncio.get_event_loop()

sentry_config = args.sentry_config
Copy link
Contributor

Choose a reason for hiding this comment

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

通过环境变量 SENTRY_URL 来设置就可以了

bin/bbox.py Outdated
Comment on lines 50 to 52
# add sentry config
parser.add_argument(
'--sentry_config',
Copy link
Contributor

Choose a reason for hiding this comment

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

用环境变量传入sentry config 即可

Copy link
Author

Choose a reason for hiding this comment

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

最新的pr没这个了。

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