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

en.chat4o.com #92

Merged
merged 25 commits into from
Dec 19, 2019
Merged

en.chat4o.com #92

merged 25 commits into from
Dec 19, 2019

Conversation

sepandhaghighi
Copy link
Owner

@sepandhaghighi sepandhaghighi commented Dec 15, 2019

Reference Issues/PRs

#91 #90

What does this implement/fix? Explain your changes.

  • 27 new 1-line art added
    1. mouse4
    2. pistols3
    3. rock on2
    4. love in my eye2
    5. love in my eye3
    6. sad3
    7. sad4
    8. headphone2
    9. cigarette3
    10. spider3
    11. barcode2
    12. equalizer
    13. boombox2
    14. surprised2
    15. playing cards
    16. playing cards hearts
    17. playing cards diamonds
    18. playing cards spades
    19. playing cards clubs
    20. spade bold
    21. heart bold
    22. diamond bold
    23. club bold
    24. spade regular
    25. heart regular
    26. diamond regular
    27. club regular
  • barcode 1-line art renamed to barcode1
  • cigaret 1-line art renamed to cigarette1
  • cigarette 1-line art renamed to cigarette2
  • headphone 1-line art renamed to headphone1
  • rock on 1-line art renamed to rock on1
  • boom box 1-line art renamed to boombox1
  • love in my eye 1-line art renamed to love in my eye1

Any other comments?

After this PR, we will be good to go for next release (version 4.4)

@sepandhaghighi sepandhaghighi added this to the art 4.4 milestone Dec 15, 2019
@codecov-io
Copy link

codecov-io commented Dec 15, 2019

Codecov Report

Merging #92 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev      #92   +/-   ##
=======================================
  Coverage   90.79%   90.79%           
=======================================
  Files           1        1           
  Lines         239      239           
  Branches       63       63           
=======================================
  Hits          217      217           
  Misses         19       19           
  Partials        3        3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 93f9813...7602192. Read the comment docs.

@sepandhaghighi
Copy link
Owner Author

sepandhaghighi commented Dec 16, 2019

UPDATE :
I made some new minor edits in README.md, INSTALL.md and travis.yml.

Note : We will drop Python 2.7 and Python 3.4 supports in 2020

art/art_dic.py Outdated
Comment on lines 282 to 284
"cigarette1": "()___)____________)",
"cigarette2": "(____((____________()~~~",
"cigarette3": "(̅_̅_̅_̅(̅_̅_̅_̅_̅_̅_̅_̅_̅̅_̅()ڪے",
Copy link
Collaborator

Choose a reason for hiding this comment

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

could you please change cigarette1 and cigarette3?
cigarette3 seems to be a more realistic model for cigarette 🚬

Copy link
Owner Author

Choose a reason for hiding this comment

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

Good point!
Done 750a601

art/art_dic.py Outdated
@@ -468,5 +468,6 @@
"surprised2": "(O_o)",
"boombox2": "♫♪ |̲̅̅●̲̅̅|̲̅̅=̲̅̅|̲̅̅●̲̅̅| ♫♪",
"equalizer": "▇ ▅ █ ▅ ▇ ▂ ▃ ▁ ▁ ▅ ▃ ▅ ▅ ▄ ▅ ▇",
"playing cards": "[♥]]] [♦]]] [♣]]] [♠]]]"
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we divide this art into 4 art?
like "playing cards hearts" : "[♥]]]" and ...

Copy link
Owner Author

Choose a reason for hiding this comment

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

Good point ;-)
I added 12 new arts for individual mode 90e5ab4 8bb2a4c
I also found this set, but I think these playing cards are too small !!

Copy link
Collaborator

Choose a reason for hiding this comment

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

I checked the link, i think these are interesting sets and worth adding, i didn't catch your point about smallness of these sets.
if it is OK for i can add this link to Font and Art suggestion

Copy link
Owner Author

Choose a reason for hiding this comment

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

I checked the link, i think these are interesting sets and worth adding, i didn't catch your point about smallness of these sets.
if it is OK for i can add this link to Font and Art suggestion

Okay, I entrust it to you ;-)

Copy link
Collaborator

@sadrasabouri sadrasabouri left a comment

Choose a reason for hiding this comment

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

I've left some comments
i hope they will be useful.

art/art_dic.py Outdated
Comment on lines 475 to 483
"playing cards clubs": "[♣]]]",
"black spade": "♠",
"black heart": "♥",
"black diamond": "♦",
"black club": "♣",
"white spade": "♤",
"white heart": "♡",
"white diamond": "♢",
"white club": "♧"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we consider a better name for these sets?
black sets are colorful in some operating systems and i think bold heart and so on and regular hear and ... or any other suitable name.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Done! 7602192

@sadrasabouri sadrasabouri merged commit 5943ef8 into dev Dec 19, 2019
@sepandhaghighi sepandhaghighi deleted the en.chat4o.com branch December 19, 2019 18:29
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.

None yet

3 participants