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

New Decoaration from smiley(#118) and webstools(#117) #127

Merged
merged 38 commits into from
May 6, 2020

Conversation

sadrasabouri
Copy link
Collaborator

@sadrasabouri sadrasabouri commented May 5, 2020

Reference Issues/PRs

#118 #117 #59

What does this implement/fix? Explain your changes.

  • Greeting GitHub actions added.
  • 190 new decorations:
    1. angry1
    2. angry2
    3. arrow1
    4. arrow2
    5. arrow3
    6. arrow4
    7. arrow5
    8. arrow6
    9. arrow7
    10. arrow8
    11. arrow_wave1
    12. arrow_wave2
    13. ball1
    14. ball2
    15. ball3
    16. ball4
    17. bazar1
    18. block1
    19. block2
    20. bow1
    21. bubble
    22. cat1
    23. cat2
    24. cat3
    25. cell1
    26. confused1
    27. confused2
    28. cross1
    29. depressed
    30. diamon3
    31. diamond1
    32. diamond2
    33. egypt1
    34. emotions1
    35. fancy1
    36. fancy10
    37. fancy11
    38. fancy12
    39. fancy13
    40. fancy14
    41. fancy15
    42. fancy16
    43. fancy17
    44. fancy18
    45. fancy19
    46. fancy2
    47. fancy20
    48. fancy21
    49. fancy22
    50. fancy23
    51. fancy24
    52. fancy25
    53. fancy26
    54. fancy27
    55. fancy28
    56. fancy29
    57. fancy3
    58. fancy30
    59. fancy31
    60. fancy32
    61. fancy33
    62. fancy34
    63. fancy35
    64. fancy36
    65. fancy37
    66. fancy38
    67. fancy39
    68. fancy4
    69. fancy40
    70. fancy41
    71. fancy42
    72. fancy43
    73. fancy44
    74. fancy45
    75. fancy46
    76. fancy47
    77. fancy48
    78. fancy49
    79. fancy5
    80. fancy50
    81. fancy51
    82. fancy52
    83. fancy53
    84. fancy54
    85. fancy55
    86. fancy56
    87. fancy57
    88. fancy58
    89. fancy59
    90. fancy6
    91. fancy60
    92. fancy61
    93. fancy62
    94. fancy63
    95. fancy64
    96. fancy65
    97. fancy66
    98. fancy67
    99. fancy68
    100. fancy69
    101. fancy7
    102. fancy70
    103. fancy71
    104. fancy72
    105. fancy73
    106. fancy74
    107. fancy75
    108. fancy76
    109. fancy77
    110. fancy78
    111. fancy79
    112. fancy8
    113. fancy80
    114. fancy81
    115. fancy82
    116. fancy83
    117. fancy84
    118. fancy85
    119. fancy86
    120. fancy87
    121. fancy88
    122. fancy89
    123. fancy9
    124. flame1
    125. flower1
    126. food1
    127. food2
    128. haha
    129. happy1
    130. happy_new_year
    131. hawaii1
    132. hawaii2
    133. heart1
    134. heart2
    135. heart3
    136. heart4
    137. heart5
    138. heart6
    139. heart7
    140. heart8
    141. heart9
    142. line1
    143. line2
    144. line3
    145. line4
    146. love_music
    147. missile1
    148. mountain1
    149. mountain2
    150. mountain3
    151. music1
    152. music2
    153. music3
    154. pencil1
    155. poker1
    156. puzzle1
    157. puzzle2
    158. puzzle3
    159. sad2
    160. sad3
    161. sad4
    162. smile1
    163. snow1
    164. snow2
    165. star10
    166. star11
    167. star12
    168. star13
    169. star14
    170. star15
    171. star16
    172. star17
    173. star3
    174. star4
    175. star5
    176. star6
    177. star7
    178. star8
    179. star9
    180. temple1
    181. title1
    182. tree1
    183. wall1
    184. wave3
    185. wave4
    186. wave5
    187. wave6
    188. wave7
    189. wave8
    190. wave9

Copy link
Owner

@sepandhaghighi sepandhaghighi left a comment

Choose a reason for hiding this comment

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

@sadrasabouri Thanks for your effort ;-)
The following decoraations are duplicated :

  1. angry2
  2. ball3
  3. ball4
  4. fancy3
  5. fancy4
  6. fancy5
  7. fancy6
  8. fancy18
  9. fancy81
  10. star12

Please keep only one from each duplicated pair

@sadrasabouri
Copy link
Collaborator Author

It seems there is a problem with happy1.

@sepandhaghighi
Copy link
Owner

It seems there is a problem with happy1.

Fixed ;-)
I will review this PR ASAP

@sadrasabouri
Copy link
Collaborator Author

sadrasabouri commented May 6, 2020

It seems there is a problem with happy1.

Fixed ;-)
I will review this PR ASAP

Thank You!

CHANGELOG.md Outdated
@@ -7,6 +7,193 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
## [Unreleased]
### Added
- `art_decor_check.py` file
- Greeting GitHub actions
- 190 new decorations:
Copy link
Owner

Choose a reason for hiding this comment

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

190 or 185 ?

Copy link
Owner

Choose a reason for hiding this comment

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

@sadrasabouri Would you please correct order in CHANGELOG.md ?
For example :

  1. fancy1
  2. fancy2
    ...
    instead of :
  3. fancy1
  4. fancy10
    ...

@sepandhaghighi sepandhaghighi merged commit a726f8d into sepandhaghighi:dev May 6, 2020
@sadrasabouri sadrasabouri mentioned this pull request May 14, 2020
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