oitsjustjose/CPSC4100
Watch
1
0
Fork
You've already forked CPSC4100
 0
Code Issues Pull requests Projects Releases Packages Activity Actions

Do we allow the fill character to be specified? #6

New issue
Closed
opened 2018-04-07 16:53:58 +00:00 by import_user_oitsjustjose_95mQ · 4 comments
import_user_oitsjustjose_95mQ commented 2018-04-07 16:53:58 +00:00 (Migrated from git.oitsjustjose.com)
Copy link

Right now the fill-character is hard-coded to be '@', should we allow this to be configurable?

Right now the fill-character is hard-coded to be '@', should we allow this to be configurable?
import_user_oitsjustjose_95mQ commented 2018-04-07 17:09:37 +00:00 (Migrated from git.oitsjustjose.com)
Copy link

Actually, per the instructions we have to allow the fill character to be provided.

I suggest we go with: fill x_cord, y_cord, fill_char.

Actually, per the instructions we have to allow the fill character to be provided. I suggest we go with: `fill x_cord, y_cord, fill_char`.
import_user_oitsjustjose_95mQ commented 2018-04-09 15:59:47 +00:00 (Migrated from git.oitsjustjose.com)
Copy link

👍 works for me.

👍 works for me.
import_user_oitsjustjose_95mQ commented 2018-04-10 16:29:14 +00:00 (Migrated from git.oitsjustjose.com)
Copy link

My python version has implemented this. I suspect yours will soon too, and the other new ones should fall suit.

My python version has implemented this. I suspect yours will soon too, and the other new ones should fall suit.
import_user_oitsjustjose_95mQ commented 2018-04-15 00:28:44 +00:00 (Migrated from git.oitsjustjose.com)
Copy link

Yup, we need fill characters. Lets do it.

Yup, we need fill characters. Lets do it.
import_user_oitsjustjose_95mQ (Migrated from git.oitsjustjose.com) closed this issue 2018-04-15 00:28:44 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
No results found.
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
good first issue

   
help wanted

   
invalid

   
question

   
wontfix
No labels
bug
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
oitsjustjose/CPSC4100#6
No description provided.