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

Backmerge: #3459 - All atom attributes should be displayed as SMARTS if at least one purely SMARTS attribute exists #3486

Conversation

AKZhuk
Copy link
Collaborator

@AKZhuk AKZhuk commented Oct 23, 2023

closes #3459
closes #3327
closes #3450
closes #3424

Check list

  • unit-tests written
  • e2e-tests written
  • documentation updated
  • PR name follows the pattern #1234 – issue name
  • branch name doesn't contain '#'
  • PR is linked with the issue
  • base branch (master or release/xx) is correct
  • task status changed to "Code review"
  • reviewers are notified about the pull request

…if at least one purely SMARTS attribute exists
@AKZhuk AKZhuk force-pushed the 3459-all-atom-attributes-should-be-displayed-as-smarts-if-at-least-one-purely-smarts-attribute-exists-1 branch from 62950be to b204bbd Compare October 23, 2023 16:01
@AKZhuk AKZhuk requested review from Nitvex and removed request for urszula-gawlowska October 24, 2023 14:48
AKZhuk and others added 2 commits October 25, 2023 11:51
…ed-as-smarts-if-at-least-one-purely-smarts-attribute-exists-1
…l-atom-attributes-should-be-displayed-as-smarts-if-at-least-one-purely-smarts-attribute-exists-1
@Nitvex Nitvex merged commit bf8677f into master Oct 25, 2023
@Nitvex Nitvex deleted the 3459-all-atom-attributes-should-be-displayed-as-smarts-if-at-least-one-purely-smarts-attribute-exists-1 branch October 25, 2023 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment