Skip to content

Comments

Added Tulsa, OK Pizza Listing #154

Open
maxxcrawford wants to merge 1 commit intostevekinney:masterfrom
maxxcrawford:tulsa
Open

Added Tulsa, OK Pizza Listing #154
maxxcrawford wants to merge 1 commit intostevekinney:masterfrom
maxxcrawford:tulsa

Conversation

@maxxcrawford
Copy link

Note – I was not able to run the $ ruby docs/map_readme.rb command. I received the following error:

./docs/map_readme.rb:38: syntax error, unexpected ':', expecting =>
        'marker-size': 'medium',
                      ^
./docs/map_readme.rb:38: syntax error, unexpected ',', expecting keyword_end
./docs/map_readme.rb:39: syntax error, unexpected ':', expecting keyword_end
        'marker-color': 'ffff00',
                       ^
./docs/map_readme.rb:39: syntax error, unexpected ',', expecting keyword_end
./docs/map_readme.rb:40: syntax error, unexpected ':', expecting keyword_end
        'marker-symbol': 'restaurant'
                        ^
./docs/map_readme.rb:41: syntax error, unexpected '}', expecting keyword_end
      },
       ^
./docs/map_readme.rb:46: syntax error, unexpected '}', expecting keyword_end

Is this me? Should I open an issue?

@selfup
Copy link
Contributor

selfup commented Aug 8, 2018

What version of ruby are you running? I think 2.2 and later support : shorthand for hashrockets 😄

@maxxcrawford
Copy link
Author

@selfup Ahhh. I was on 2.0. Updated and then received the #153 error:

Google API error: over query limit.

So, I still haven't ran the command successfully.

@selfup
Copy link
Contributor

selfup commented Aug 8, 2018

Well currently that is some good progress 🙏

I guess we now have to wait for the API to no longer exceed it's quoted API call limit.

Unclear when that refreshes.

@stevekinney You know when the API limit refreshes? or is it hourly?

@maxxcrawford
Copy link
Author

@selfup @stevekinney I've been running the command randomly since last week and keep getting the same limit error.

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