added the fullformatedtime capability#195
added the fullformatedtime capability#195Saad-Imtiaz wants to merge 4 commits intoarduino-libraries:masterfrom
Conversation
|
Memory usage change @ 73ab371
Click for full report table
Click for full report CSV |
per1234
left a comment
There was a problem hiding this comment.
Thanks for your pull request @Saad-Imtiaz. I see we have a previous proposal for the addition of this getFormattedDate function: #119
The difference in your proposal is that the function takes an argument that allows it to be used as a converter for arbitrary input. I feel that such a conversion functionality is out of scope for this library and would be better provided by a library dedicated to such things (perhaps even just using strftime directly).
So I will close this as essentially a duplicate of #119
By the way, I noticed that you introduced a huge quantity of unrelated and harmful changes in addition to the reasonable proposal of the addition of the getFormattedDate function. Those unrelated changes made the pull request very difficult to evaluate and would not have been accepted even if the primary proposal was accepted. I recommend you take more care in the future when you make contributions to open source software projects. You should carefully check the diff to make sure your commits only make the intended changes.
One of the serious problems in this pull request is that you changed the line endings in some files from LF to CRLF. LF is generally the standard line ending in any software project. You should check the settings of your text editor/IDE to see if it is configured to use CRLF so that you can avoid this problem in the future.
Add the Google API format that is 2013-02-14T13:15:03.
This format type can be accessed by using fullformatedtime().