semantic-kernel
semantic-kernel copied to clipboard
fixed missing today() in TimeSkill
Motivation and Context
the method Today() in TimeSkill.cs and today() in TimeSkill.py was missing. Today must be an alias for Date.
Description
Add the method Today.
Contribution Checklist
- [x] The code builds clean without any errors or warnings
- [x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
- [x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with
dotnet format
- [x] All unit tests pass, and I have added new tests where possible
- [ ] I didn't break anyone :smile:
Instead of
[SKFunction("Get the current date")]
public string Today()
{
// Example: Sunday, 12 January, 2025
return this.Date();
}
You could just write
[SKFunction("Get the current date")]
public string Today() => Date();
that's better one.
@lemillermicrosoft no change in any .md file, please help to pass markdown-link-check