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.

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

v10.4.4 #2322

Merged
merged 46 commits into from
Oct 9, 2024
Merged

v10.4.4 #2322

merged 46 commits into from
Oct 9, 2024

Conversation

ouziel-slama
Copy link
Contributor

@ouziel-slama ouziel-slama commented Oct 9, 2024

  • Double-check the spelling and grammar of all strings, code comments, etc.
  • Double-check that all code is deterministic that needs to be
  • Add tests to cover any new or revised logic
  • Ensure that the test suite passes
  • Update the project release notes
  • Update the project documentation, as appropriate, with a corresponding Pull Request in the Documentation repository

Merge pull request #2308 from CounterpartyXCP/develop
else:
with start_sentry_span(op="cache.get") as sentry_get_span:
sentry_get_span.set_data("cache.key", cache_key)
if cache_key in BLOCK_CACHE:

Check warning

Code scanning / pylint

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it. Warning

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it.


def is_server_ready():
# TODO: find a way to mock this function for testing

Check warning

Code scanning / pylint

TODO: find a way to mock this function for testing. Warning

TODO: find a way to mock this function for testing.

def refresh_current_block(db):
# update the current block index
global CURRENT_BLOCK_TIME # noqa F811

Check warning

Code scanning / pylint

Using the global statement. Warning

Using the global statement.


def refresh_backend_height(db, start=False):
global BACKEND_HEIGHT, BACKEND_HEIGHT_TIMER # noqa F811

Check warning

Code scanning / pylint

Using the global statement. Warning

Using the global statement.
refresh_backend_height(timer_db, start=True)
else:
refresh_current_block(timer_db)
global BACKEND_HEIGHT # noqa F811

Check warning

Code scanning / pylint

Using the global statement. Warning

Using the global statement.
self.cfg.post_fork(self, worker)
worker.init_process()
sys.exit(0)
except SystemExit:

Check warning

Code scanning / pylint

The except handler raises immediately. Warning

The except handler raises immediately.
self.log.debug("Exception while loading the application", exc_info=True)
sys.stderr.flush()
sys.exit(self.APP_LOAD_ERROR)
except Exception:

Check warning

Code scanning / pylint

Catching too general exception Exception. Warning

Catching too general exception Exception.
try:
worker.tmp.close()
self.cfg.worker_exit(self, worker)
except Exception:

Check warning

Code scanning / pylint

Catching too general exception Exception. Warning

Catching too general exception Exception.
pass


class GunicornApplication(gunicorn.app.base.BaseApplication):

Check warning

Code scanning / pylint

Method 'init' is abstract in class 'BaseApplication' but is not overridden in child class 'GunicornApplication'. Warning

Method 'init' is abstract in class 'BaseApplication' but is not overridden in child class 'GunicornApplication'.
class GunicornApplication(gunicorn.app.base.BaseApplication):
def __init__(self, app, args=None):
self.options = {
"bind": "%s:%s" % (config.API_HOST, config.API_PORT),

Check warning

Code scanning / pylint

Formatting a regular string which could be an f-string. Warning

Formatting a regular string which could be an f-string.
@adamkrellenstein adamkrellenstein merged commit 679ab89 into master Oct 9, 2024
18 of 19 checks passed
ouziel-slama added a commit that referenced this pull request Oct 16, 2024
Merge pull request #2322 from CounterpartyXCP/develop
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.

3 participants